Open
Conversation
added 27 commits
October 16, 2020 17:37
add status flag, EventStatus
display cancelled events' title strikethroughed
fix typo
add a new db operation, changeEventStatusCodeById, which changes event's status to a given status code.
add a new api, changeEventStatus, which updates event's status to a given status code
fix typo
create a new file with a function that helps to display confirmation box
fix typo
use show_modal() to display the confirmation box
add logistics for cancel/uncancell event
move moda/confirmation box to a global scope (inside head.ejs)
include modal.js
move modal/confirmation box from event
not sure if this should be changed. When I am testing email service, with path = /app/.., file not exists error was thrown.
jialuogao
requested changes
Nov 15, 2020
Collaborator
jialuogao
left a comment
jialuogao
left a comment
There was a problem hiding this comment.
I was not able to see majority of your pages, please fix those issues I found and let me know then I'll take a look later
added 2 commits
November 19, 2020 01:45
add a new api for checking if user has profile
add an if statement to check db error
added 3 commits
November 19, 2020 01:46
add a guard to check if user has profile.
fix the msg issue
revert back
klmui
previously approved these changes
Apr 26, 2021
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
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.
Users can follow events and receive an email of the event.
Followed events can be seen together in a list view,
/event/followed.ejs.An email is going to send to the user's email if the
RSVPbutton is clicked.I have modified the
notification-bara bit. Allowing it to display customized failure/success message. Also, when success, thebackground-colorof thenotification-baris green, whereas when failure, thebackground-coloris red.