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

Bugfix: Force refresh on save when event data is updated #107

Open
h0ethan04 opened this issue Mar 8, 2025 · 0 comments
Open

Bugfix: Force refresh on save when event data is updated #107

h0ethan04 opened this issue Mar 8, 2025 · 0 comments
Assignees
Labels

Comments

@h0ethan04
Copy link
Collaborator

Description

Bug created by not elevating state of the modal high enough. Modify event functionality works, except we don't reload components correctly when a change is committed.
Look into moving the useStates into the parent level, so that all components from the main page, the card, and the different modals all have the correct values displayed.
There's also a few missing data values, make sure all values are present where expected.

READ ALL INSTRUCTIONS BEFORE STARTING

  • Note:
    • When working on the frontend, we want to work from making sure the site is responsive starting from a mobile device (this will make it easier when we modifying the styling for bigger screens)
    • When checking how the frontend looks on a mobile device through chrome's inspect element, be sure to select iPhone SE for test!!!
  • Heads up:
    • Feel free to utilize /client/src/components/playground/Playground.jsx to test out anything before putting it in their respective pages, or if there currently is no respective page

    • When prompted to log in, you can sign in with the account:

    • teacher:

    • admin:

Instructions

  • Bugfix: elevate useStates for the modal such that they maintained by the page; this should resolve the problem. Make sure you pass props through the components correctly.
  • Bugfix: there is a column in the events table for capacity, update the cards, forms, and modals for teacher view to reflect this change.

Notes

  • Discovery and Bookings have been combined for teacher view, check that all props have been passed through to the cards properly

Acceptance Criteria

  • Resolves all bugs without introducing new unintended behaviors.
  • Matches hi-fi (but you can ignore the colored components for now)
  • all missing columns have been added
  • modal and all relevant cards refresh when saving changes to an event

Resources

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants