Skip to content
New issue

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

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

Already on GitHub? # to your account

Encrypt Patient Id for EGASP upload. #321

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

Conversation

9sneha-n
Copy link
Contributor

@9sneha-n 9sneha-n commented Feb 6, 2025

📌 References

📝 Implementation

  1. Fetch the secret key and nonce from dhis
  2. Encrypt Patient Id on EGASP upload

IMPORTANT NOTE
The following changes have to released with metadata changes to migrate AMR_GLASS_EGASP_PRE_ENCRYPTION_KEY program and also we need to create an event with secret key and nonce values.

📹 Screenshots/Screen capture

encryption-1.mov
encryption-3.mov
encryption-2.mov

🔥 Testing

@ifoche
Copy link
Member

ifoche commented Feb 7, 2025

@9sneha-n 9sneha-n marked this pull request as ready for review February 7, 2025 06:30
@@ -0,0 +1,9 @@
import { FutureData } from "../entities/Future";

export type EncryptionData = {
Copy link
Contributor

Choose a reason for hiding this comment

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

This is not a domain entity?

Copy link
Contributor

@anagperal anagperal left a comment

Choose a reason for hiding this comment

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

thanks @9sneha-n ! Only one comment that does not affect to functionality

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

Successfully merging this pull request may close these issues.

3 participants