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

fix: send proper email change token in hook #1613

Closed
wants to merge 1 commit into from

Conversation

J0
Copy link
Contributor

@J0 J0 commented Jun 6, 2024

What kind of change does this PR introduce?

Token field should send otp and TokenNew should send otpNew

@coveralls
Copy link

coveralls commented Jun 6, 2024

Pull Request Test Coverage Report for Build 9408603976

Details

  • 0 of 2 (0.0%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 57.561%

Changes Missing Coverage Covered Lines Changed/Added Lines %
internal/api/mail.go 0 2 0.0%
Totals Coverage Status
Change from base Build 9404687053: 0.0%
Covered Lines: 8507
Relevant Lines: 14779

💛 - Coveralls

@J0
Copy link
Contributor Author

J0 commented Jun 10, 2024

Addressed in #1614

@J0 J0 closed this Jun 10, 2024
@J0 J0 deleted the j0/patch_secure_email_change branch November 21, 2024 08:09
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