Skip to content
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.

Sign up for GitHub

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

Jump to bottom

BoG Changes Spring 23 #7

Merged
merged 72 commits into from
Apr 2, 2024
Merged

BoG Changes Spring 23 #7

merged 72 commits into from
Apr 2, 2024

Conversation

samarth52
Copy link
Contributor

Merges #3, #5, and #6
Adds support to:

  • Handle friend invitations being accepted
  • Regularly clear unused friend invitations (1 week expiry)
  • Delete sender data from a friend's FriendData record
  • Delete friend data from a sender's ScheduleData record
  • Send emails, as well as create email templates

yatharth-b and others added 20 commits March 1, 2023 16:36
@yatharth-b
add code to handle expired invites and add more functionality to crea…
ce7f95f 
…te and handle invitation
@yatharth-b
update method of getting all documents
7dab187
@yatharth-b
remove invite from sender's schedule if expired
2def951
@yatharth-b
comments
f888637
@yatharth-b
should be able to handle both fetch and axios
3f50209
@yatharth-b
nits
8e39777
@yatharth-b
nits
d34acae
@samarth52
nit
603f7b8
@EmilyAL001
Email Template Styling
7db21c6
@samarth52
added function to delete friend and updated types
73860b3
@samarth52
Merge branch 'main' of https://github.com/gt-scheduler/firebase-conf …
a095ce5 
…into Samarth/delete-invitation
@samarth52
lint and prettier
53393f5
@samarth52
Merge branch 'yatharth/173-174' of https://github.com/gt-scheduler/fi…
8ad1871 
…rebase-conf into bog-changes-s23-merged
@samarth52
Merge branch 'emily/206-email-template' of https://github.com/gt-sche…
5f15e3d 
…duler/firebase-conf into bog-changes-s23-merged
@samarth52
Merge branch 'Samarth/delete-invitation' of https://github.com/gt-sch…
3240045 
…eduler/firebase-conf into bog-changes-s23-merged
@samarth52
updated node version
50ed45b
@samarth52
Updated request body parsing
412466e
@samarth52
Handle friend invitation accounts for valid friendRecord
a0f3b11
@samarth52
Update friend info if not exists
7ec1957
@samarth52
Added function to delete friend data from sender modal
cceb918
@samarth52 samarth52 requested a review from nhatnghiho October 20, 2023 07:16
@samarth52 samarth52 mentioned this pull request Jan 16, 2024
Merged
aeluro1 and others added 27 commits February 29, 2024 18:44
@aeluro1
Renamed invitation deletion function
c701a91
@aeluro1
Updated logic for locating shared schedule requiring deletion
693928a
@aeluro1
Deleted obsolete function
82d777b
@aeluro1
Minor fix
e52e1f1
@aeluro1
Update type
0f20ab0
@aeluro1
Implemented feedback
f16d879
@aeluro1
Fixes
24ef168
@yatharth-b
error codes for handling invites
1508f5a
@aeluro1
temp check in
3d11996
@aeluro1
Rewrote deletion database calls to utilize atomic transaction
507ac74
@aeluro1
Added debugging scripts
bd8f152
@aeluro1
Merge branch 'bog-changes-s23-merged' into delete-shared-schedules
42533f3
@aeluro1
Fixes
6c368dc
@aeluro1
nit
1ab2ba5
@aeluro1
Remove debug
6a8784c
@yatharth-b
only intended user can accept invites
a16373f
@yatharth-b
fixes
a26a38c
@yatharth-b
shift to east
0f767b1
@yatharth-b
shift to east
e48994a
@yatharth-b
Merge branch 'bog-changes-s23-merged' of https://github.com/gt-schedu…
ab4d02f 
…ler/firebase-conf into bog-changes-s23-merged

:qa!
:qa!
@yatharth-b
nit
aabbd39
@yatharth-b
prettier
9649933
@yatharth-b
Merge pull request #15 from gt-scheduler/delete-shared-schedules
ea335fd 
Consolidated shared schedule deletion functions
@yatharth-b
error messages + send term to client + check for if user has already …
d719efa 
…accepted versions
@yatharth-b
run prettier
efcdb82
@yatharth-b
email styling
4bb7a37
@yatharth-b
email styling
11cb773
yatharth-b
yatharth-b approved these changes Apr 2, 2024
View reviewed changes
Copy link
Contributor

@yatharth-b yatharth-b left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@yatharth-b yatharth-b merged commit 34dc330 into main Apr 2, 2024
1 check passed
yatharth-b added a commit to gt-scheduler/website that referenced this pull request Apr 2, 2024
@samarth52 @nhatnghiho
@EmilyAL001 @sophiazlin @nathangong @yatharth-b @iphonecats @aeluro1 @Nathan-Papa @joannacheng21
BoG Spring 24 Deployment - Friends Feature (#253)
bb9b7b7 
Complete friends feature frontend.

cloud functions : gt-scheduler/firebase-conf#7

---------

Co-authored-by: Hailey Ho <[email protected]>
Co-authored-by: Hailey Ho <[email protected]>
Co-authored-by: Nghi Ho <[email protected]>
Co-authored-by: EmilyAL001 <[email protected]>
Co-authored-by: Sophia Lin <[email protected]>
Co-authored-by: Nathan Gong <[email protected]>
Co-authored-by: Yatharth Bhargava <[email protected]>
Co-authored-by: Hailey Ho <[email protected]>
Co-authored-by: Hailey Ho <[email protected]>
Co-authored-by: Hailey Ho <[email protected]>
Co-authored-by: Hailey Ho <[email protected]>
Co-authored-by: Yatharth Bhargava <[email protected]>
Co-authored-by: Hailey Ho <[email protected]>
Co-authored-by: Hailey Ho <[email protected]>
Co-authored-by: EmilyAL001 <[email protected]>
Co-authored-by: nhatnghiho <[email protected]>
Co-authored-by: Brandon Ho <[email protected]>
Co-authored-by: aeluro1 <[email protected]>
Co-authored-by: Nathan Papa <[email protected]>
Co-authored-by: Joanna Cheng <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@yatharth-b yatharth-b yatharth-b approved these changes

@nhatnghiho nhatnghiho Awaiting requested review from nhatnghiho

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@samarth52 @yatharth-b @EmilyAL001 @joannacheng21 @aeluro1