Conversation
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the ✨ Finishing Touches🧪 Generate unit tests
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. CodeRabbit Commands (Invoked using PR/Issue comments)Type Other keywords and placeholders
CodeRabbit Configuration File (
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #772 +/- ##
=========================================
Coverage 30.67% 30.67%
Complexity 819 819
=========================================
Files 48 48
Lines 3971 3971
=========================================
Hits 1218 1218
Misses 2753 2753 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
I think this is incorrect and should be split into 2 messages. if unconfirmed or if no email otherwise if no email at all a default message is displayed on Special:ConfirmEmail that isnt really adding a new one, which is really why it is currently set to ChangeEmail I believe |
The current email confirmation link is
Special:ChangeEmail, which has caused new users to think that they must change their email to confirm it; changing this toSpecial:ConfirmEmailwill eliminate a lot of confusion.