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.

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

Event follow and RSVP #38

Open
wants to merge 55 commits into
base: dev
Choose a base branch
from
Open

Event follow and RSVP #38

wants to merge 55 commits into from

Conversation

kitkatdafu
Copy link
Collaborator

@kitkatdafu kitkatdafu commented Nov 14, 2020

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 RSVP button is clicked.

I have modified the notification-bar a bit. Allowing it to display customized failure/success message. Also, when success, the background-color of the notification-bar is green, whereas when failure, the background-color is red.

Reid Chen 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.
Copy link
Collaborator

@jialuogao jialuogao left a comment

Choose a reason for hiding this comment

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

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

routes/services/email_service.js Outdated Show resolved Hide resolved
server/user_db.js Outdated Show resolved Hide resolved
views/template/nav.ejs Outdated Show resolved Hide resolved
Yi Chen 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
Yi Chen added 3 commits November 19, 2020 01:46
add a guard to check if user has profile.
fix the msg issue
@kitkatdafu kitkatdafu requested a review from jialuogao November 19, 2020 07:54
klmui
klmui previously approved these changes Apr 26, 2021
@klmui klmui self-requested a review April 26, 2021 21:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants