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: verify status edit page #1521

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

Chisomchima
Copy link
Member

Description:

This PR implements the functionality for indicating the email verification status on the User Management Page of the User App. The UI behavior is based on the system version and email configuration. It ensures that the email verification status for users is displayed when relevant, in accordance with the acceptance criteria.

Changes Made:

New User Page:

  • No email verification messaging is displayed when creating a new user, as specified in the acceptance criteria.

Editing User Page:

  • For v41 or earlier versions of the system:

    • There are no changes to the UI related to email verification status.
  • For v42 or later versions:

    • When email is not configured (indicated by the emailConfigured value in systemInfo):

      • There are no changes to the UI related to email verification.
    • When email is configured:

      • If the user has verified their email:

        • The UI displays a message with green color (green600) and <IconCheckmarkCircle16>, indicating that the email is verified.
      • If the user has not verified their email:

        • If the system requires email verification (enforceVerifiedEmail: true):

          • A red warning message is displayed with <IconWarning16>, indicating that the email is not verified.
        • If the system does not require email verification:

          • A neutral message is displayed with the default theme color and <IconInfo16>, indicating that the email has not been verified.

Email Verification Status:

  • The emailVerified property is now used from the user object to determine whether the email has been verified.
  • The UI is updated accordingly to display the appropriate icon and message, based on the system settings:
    • enforceVerifiedEmail in api/systemSettings determines whether email verification is required by the system.

@Chisomchima Chisomchima marked this pull request as draft January 15, 2025 15:33
@Chisomchima Chisomchima requested a review from tomzemp January 15, 2025 15:33
@dhis2-bot
Copy link
Contributor

dhis2-bot commented Jan 15, 2025

🚀 Deployed on https://pr-1521--dhis2-user.netlify.app

@dhis2-bot dhis2-bot temporarily deployed to netlify January 15, 2025 15:33 Inactive
@Chisomchima Chisomchima marked this pull request as ready for review January 16, 2025 13:38
@dhis2-bot dhis2-bot temporarily deployed to netlify January 16, 2025 13:39 Inactive
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