-
Notifications
You must be signed in to change notification settings - Fork 9
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
Jiawei/recruitment/eventpage #95
Conversation
Hi, good attempt. No worries, if you need help, feel free to reach out to Michael, me or Jess. Try to fix the merge conflicts via rebase (or merge if you are not confident) first |
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.
Some changes required and also resolving the merge conflicts
Hi @michaelseyo and @woowenjun99 I have made the changes and resolved the conflicts, hope the PR is fine this time. Thanks. |
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.
Just a small change to be made but other than that looks good, try and do yarn run build
to see if you're able to build.
okay should be good @michaelseyo |
Nice ok thanks @Kappaccinoh :) |
90d06c1
to
6fb433b
Compare
Description
Add edit and delete functions to event page
Fixes # (issue)
Reorganised router file structure, separated attendance and event
Pending final UI updates from Dilys for Button or Popup UI
Type of change
Please delete options that are not relevant.
Checklist: