fix: Check for user account status before attemping to process email change. #2298

Merged
Jade merged 1 commit from Clubs/continuwuity:2231-change-email-check into main 2026-09-29 18:36:36 +00:00
Contributor

This pull request...

  • Adds a check to !admin users change-email to verify that a user exists

Fixes: #2231

Pull request checklist:

  • This pull request targets the main branch, and the branch is named something other than
    main.
  • I have written an appropriate pull request title and my description is clear.
  • I understand I am responsible for the contents of this pull request.
  • I have followed the contributing guidelines:
This pull request... - Adds a check to !admin users change-email to verify that a user exists Fixes: #2231 **Pull request checklist:** - [x] This pull request targets the `main` branch, and the branch is named something other than `main`. - [x] I have written an appropriate pull request title and my description is clear. - [x] I understand I am responsible for the contents of this pull request. - I have followed the [contributing guidelines][c1]: - [x] My contribution follows the [code style][c2], if applicable. - [x] I ran [pre-commit checks][c1pc] before opening/drafting this pull request. - [x] I have [tested my contribution][c1t] (or proof-read it for documentation-only changes) myself, if applicable. This includes ensuring code compiles. - [x] My commit messages follow the [commit message format][c1cm] and are descriptive. [c1]: https://forgejo.ellis.link/continuwuation/continuwuity/src/branch/main/CONTRIBUTING.md [c2]: https://forgejo.ellis.link/continuwuation/continuwuity/src/branch/main/docs/development/code_style.mdx [c1pc]: https://forgejo.ellis.link/continuwuation/continuwuity/src/branch/main/CONTRIBUTING.md#pre-commit-checks [c1t]: https://forgejo.ellis.link/continuwuation/continuwuity/src/branch/main/CONTRIBUTING.md#running-tests-locally [c1cm]: https://forgejo.ellis.link/continuwuation/continuwuity/src/branch/main/CONTRIBUTING.md#commit-messages
fix: check for user account status before attemping to process email change.
Some checks failed
Auto Labeler / Apply labels based on changed files (pull_request_target) Successful in 3s
Checks / Changelog / Check changelog is added (pull_request_target) Successful in 6s
Documentation / Build and Deploy Documentation (pull_request) Has been skipped
Checks / Prek / Check changed files (pull_request) Successful in 5s
Checks / Prek / Pre-commit & Formatting (pull_request) Successful in 1m56s
Checks / Prek / Clippy and Cargo Tests (pull_request) Failing after 6m1s
b99661eed5
ginger approved these changes 2026-09-28 23:35:54 +00:00
Jade approved these changes 2026-09-29 17:45:33 +00:00
Jade left a comment

Thanks for the contribution!

Thanks for the contribution!
Jade scheduled this pull request to auto merge when all checks succeed 2026-09-29 17:47:40 +00:00
Jade merged commit 684820c7c1 into main 2026-09-29 18:36:36 +00:00
Jade deleted branch 2231-change-email-check 2026-09-29 18:36:36 +00:00
Sign in to join this conversation.
No reviewers
No milestone
No assignees
3 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
continuwuation/continuwuity!2298
No description provided.