Updated confirmation emails to use the same jitsi meeting for mentors… #117
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.
… and mentees
Pull Request Template
1. Targeted Issue
Confirmation emails were sending a different jitsi meeting to mentors and mentees.
Issue #90
2. Overview of Solution
Instead of creating a meeting in models.py for the mentor/mentee emails, we moved it to views.py to create a single meeting. We called the create_meeting_link() function and passed that link into the functions to send each email.
3. Tools Used
Python, Django
4. Testing Strategy
Set up new session to check if both users got the same meeting invite
5. Future Implications
n/a
6. Screenshots
n/a
7. Code Reviewers
@GitLukeW
Before you go...
Big high-fives for your incredible pull request! We're thrilled to have you as a contributor! Our team will dig into your proposed changes and give you some feedback or merge them if they're good to go. Your contributions mean the world to us!