Skip to content

Conversation

Diana20920
Copy link
Contributor

Summary

  • This work is behind a feature toggle (flipper): NO
    • The controller what will be added and will be calling this service will use the existing feature toggle (flipper)

This adds the EmailVerificationService, which contains the core logic for verifying a user’s email. The controller calls this service to start and complete the verification process, and the service triggers emails to the user when the process begins and when it completes successfully. Logging has also been added for visibility. Remaining components of this work will be integrated later, and this service may be updated as needed.

  • AuthenticatedExperience Team

Related issue(s)

Testing done

  • New code is covered by unit tests

Acceptance criteria

  • I fixed|updated|added unit tests and integration tests for each feature (if applicable).
  • No error nor warning in the console.
  • Events are being sent to the appropriate logging solution
  • Documentation has been updated (link to documentation)
  • No sensitive information (i.e. PII/credentials/internal URLs/etc.) is captured in logging, hardcoded, or specs
  • Feature/bug has a monitor built into Datadog (if applicable)
  • If app impacted requires authentication, did you login to a local build and verify all authenticated routes work as expected
  • I added a screenshot of the developed feature

@va-vsp-bot
Copy link
Collaborator

Error: A file (or its parent directories) does not have a CODEOWNERS entry. Please update the .github/CODEOWNERS file and add the entry for the Offending file: app/services/email_verification_service.rb

@Diana20920 Diana20920 force-pushed the 119247-email-verification-service-layer branch from 0c29af5 to dba68f9 Compare October 20, 2025 20:36
@va-vsp-bot
Copy link
Collaborator

Error: A file (or its parent directories) does not have a CODEOWNERS entry. Please update the .github/CODEOWNERS file and add the entry for the Offending file: lib/email_verification/jwt_generator.rb

@Diana20920 Diana20920 force-pushed the 119247-email-verification-service-layer branch from dba68f9 to 38752f2 Compare October 21, 2025 16:01
@Diana20920 Diana20920 requested a review from micahaspyr October 21, 2025 16:39
@Diana20920 Diana20920 force-pushed the 119247-email-verification-service-layer branch 3 times, most recently from c99f885 to f5123a5 Compare October 21, 2025 19:57
@Diana20920 Diana20920 force-pushed the 119247-email-verification-service-layer branch from f5123a5 to 5000582 Compare October 22, 2025 04:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants