Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: event delivery #193

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

feat: event delivery #193

wants to merge 2 commits into from

Conversation

JohanGHole
Copy link
Collaborator

Added support for event delivery from the event queue to DHIS2. The structure of the route builder is heavily based on the report delivery route builder, as I discovered that I could adopt a similar logical structure. The core modifications involved adjusting the API endpoints and ensuring compatibility with DHIS2's requirements for event data.

Key Changes:

  • Implement event delivery route builder
  • Ensured that the payload formatting meets DHIS2's specifications for event data
  • Added aggregation strategies for aggregating event payload from DHIS2
  • Added aggregation strategy for extracting program stage data element codes to be compared with data element codes from RapidPro flow.
  • Added datasonnet transformation
  • Test coverage

@JohanGHole JohanGHole self-assigned this Mar 13, 2024
@JohanGHole JohanGHole requested a review from cjmamo March 13, 2024 14:23
@cjmamo
Copy link
Contributor

cjmamo commented Mar 18, 2024

Does any documentation need to added for this feature? If yes, please include it in this PR.

Copy link
Contributor

@cjmamo cjmamo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

See comment about docs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants