-
Notifications
You must be signed in to change notification settings - Fork 6
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
Feat: confirmation emails for joining organization #962
Feat: confirmation emails for joining organization #962
Conversation
Codecov Report
@@ Coverage Diff @@
## master #962 +/- ##
==========================================
+ Coverage 65.49% 66.80% +1.30%
==========================================
Files 131 144 +13
Lines 5655 5982 +327
Branches 1314 1409 +95
==========================================
+ Hits 3704 3996 +292
- Misses 1819 1851 +32
- Partials 132 135 +3
Flags with carried forward coverage won't be shown. Click here to find out more. see 179 files with indirect coverage changes 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
…or email title and html body
….com:rage/mooc.fi into feat-add-user-organization-join-confirmation
61752fb
to
cc909b7
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
CodeQL found more than 10 potential problems in the proposed changes. Check the Files changed tab for more details.
Even if the main feature of this PR will not go into use (yet) and is not indeed ready for production, this branch lived for so long and eventually veered so away from master that it's best to merge this to keep master up to date. |
Add migrations and graphql stuff for join confirmations for organization memberships