Added account confirmation emails for email registrations "fixes #3896" #3903
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Merge dev to staging
Merge dev into staging
merge staging into master
Pushes development to staging
Pushing staging to master
fixes try method on WET activity views
Pushes develop to staging
Pushing develop to staging
pushes develop to staging
pushes staging to master
pushes develop to staging
pushing staging to production
Pushes development to staging
Pushes FIX code from develop to staging
Pushes staging to production
pushes development to staging
pushes development to staging
pushes develop to staging
pushes staging to production
* Update README.md * March 2022 Update (#3799) * updates promo texts removes casettes * updates gh commits feature * fixes and disables failing features * disables commit count feature * updates welcome mailer * removes Slacj invite job due to deprecated method Co-authored-by: Thomas Ochman <[email protected]> Co-authored-by: Thomas Ochman <[email protected]>
…the :confired_at for existing users and wrote feature spec confirm_user_email.feature
mattlindsey
reviewed
Aug 15, 2023
.gitignore
Outdated
| @@ -102,8 +102,12 @@ certbot.log | |||
| rerun.txt | |||
|
|
|||
| # Ignore VCR | |||
| <<<<<<< HEAD | |||
There was a problem hiding this comment.
These lines with "<<<" and "===" mean that there was a merge conflict in your changes.
I don't think this file needs to be modified for this PR, so can you restore it?
mattlindsey
reviewed
Aug 15, 2023
README.md
Outdated
| ``` | ||
|
|
||
| Updating the pages requires the administrator to run the `rake fetch_github:content_for_static_pages` task. | ||
| ## Walkthroughs |
There was a problem hiding this comment.
Is this change from an old version of this file? I don't think this file needs to be changed for this PR.
mattlindsey
reviewed
Aug 15, 2023
| @@ -0,0 +1,5 @@ | |||
| class RemoveReconfirmableFromUsers < ActiveRecord::Migration[7.0] | |||
There was a problem hiding this comment.
If this column isn't needed, you can delete the migration that adds it and the one that removes it.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Using devise :confirmable I added email confirmation
Feature spec in features/users/confirm_user_email.feature