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

DQ: Get Contact Emails for Follow-Up issue #27

Merged
merged 9 commits into from
Mar 18, 2024

Conversation

eperedo
Copy link
Contributor

@eperedo eperedo commented Mar 16, 2024

📌 References

📝 Implementation

📹 Screenshots/Screen capture

Module 1

contact_emails_module1.webm

Module 2

contact_emails_module2.webm

🔥 Notes to the tester

  • I've created this PR as a draft because it depends on this Add Filters for Issue Listing #26

  • When I was testing this I noticed that adding a user to an userGroup is not an immediate change, sometimes it takes almost 2 minutes to these changes to be in the /api/userGroups/{id} endpoint.

  • Metadata attached in issue: https://app.clickup.com/t/86941gkg3 or pull changes from d2-docker docker.eyeseetea.com/eyeseetea/dhis2-data:2.38-widp-hwf-dq

#8693r68h2

@eperedo eperedo requested a review from Ramon-Jimenez March 16, 2024 17:06
@ifoche
Copy link
Member

ifoche commented Mar 16, 2024

Copy link
Contributor

@Ramon-Jimenez Ramon-Jimenez left a comment

Choose a reason for hiding this comment

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

Working great, thanks @eperedo

@eperedo eperedo marked this pull request as ready for review March 18, 2024 12:06
@Ramon-Jimenez Ramon-Jimenez merged commit 1496c53 into development Mar 18, 2024
1 check passed
@Ramon-Jimenez Ramon-Jimenez deleted the feat/contact-emails branch March 18, 2024 12:49
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.

3 participants