Skip to content

Add Admin-view Panel/Dashboard and Register logic #2107

Open
KhoaNguyen706 wants to merge 1 commit intodevfrom
khoa/admin-view
Open

Add Admin-view Panel/Dashboard and Register logic #2107
KhoaNguyen706 wants to merge 1 commit intodevfrom
khoa/admin-view

Conversation

@KhoaNguyen706
Copy link
Copy Markdown
Collaborator

Admin of the event can see the list of attendee now!!!!

-Dashboard when failed to load attendee:
image

-Admin of the event have a button to see the list of attendee right now:
image

-Dashboard when successfully fetch the list of attendee:
image

  • Member and Not Admin of event only register. If register successfully -> they can no longer register again.
image

Comment thread docker/Dockerfile Outdated
Copy link
Copy Markdown
Contributor

@adarshm11 adarshm11 left a comment

Choose a reason for hiding this comment

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

what happened to the docker-compose file......

Copy link
Copy Markdown
Collaborator

@weslayer weslayer left a comment

Choose a reason for hiding this comment

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

ummmm

Copy link
Copy Markdown
Collaborator

@steeevin88 steeevin88 left a comment

Choose a reason for hiding this comment

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

there's some changes that should not be here... let's remove them as per comments

please actually review your changes before you push a PR lol, i'm sure you would have caught a lot of these first. obv this is a club setting so it's not super serious but if you push unreviewed code for a company, your mentors would be very unhappy 😅

let's also rebase again, i think you missed a few changes

Comment thread docker/Dockerfile Outdated
Comment thread src/APIFunctions/SCEvents.js Outdated
Comment thread docker-compose.yml
Comment thread docker-compose.yml Outdated
Comment thread docker-compose.yml Outdated
Comment thread docker-compose.yml Outdated
Comment thread docker-compose.yml Outdated
Comment thread docker-compose.yml Outdated
Comment thread nginx.conf Outdated
- Reset branch onto origin/dev; keep dev Events list/registration pages
- Add EventAttendeesDashboard, route /events/:id/admin/attendees
- Extend SCEvents API client (registrations, registerForEvent, registerForSCEvent alias, getMyEventRegistrationState) and improve fetch error payloads
- CalendarView: admin attendees link, registration state for published events
- Gate Create/Edit event pages and admin nav SCEvents links on config.SCEvents.ENABLED
- UserNavbar: optional Events link when SCEvents enabled

Made-with: Cursor
Copy link
Copy Markdown
Collaborator

@steeevin88 steeevin88 left a comment

Choose a reason for hiding this comment

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

can you update the screenshots? it doens't look like it's the correct admin view, we should see the admin sidebar + not the regular navbar

{ title: 'About', route: '/about' },
{ title: 'Projects', route: '/projects' },
{ title: 'Summer Internship', route: '/s/internship', newTab: true },
...(config.SCEvents?.ENABLED ? [{ title: 'Events', route: '/events' }] : []),
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

this shouldn't be here yet + the config you are using is outdated

{ title: 'Printing', route: '/2DPrinting' },
{ title: 'Chat', route: '/messaging' },
{ title: 'LED Sign', route: '/led-sign' },
...(config.SCEvents?.ENABLED ? [{ title: 'Events', route: '/events' }] : []),
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

same here

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.

5 participants