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: Drafts in Bookings (Teacher View) #112

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

Bugfix: Drafts in Bookings (Teacher View) #112

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

Comments

@h0ethan04
Copy link
Collaborator

Description

Bookings page doesn’t handle drafts correctly – saving changes for drafts should have option to publish them, not just update the draft

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:

    • student:

    • teacher:

    • admin:

Instructions

  • for teacher view on the bookings page, there should be a "draft" tab. All the cards in here need to open their respective createEvent/createClass forms and properly send put requests when information is saved.
  • additionally, add to the forms a button allowing to publish, which will then flip the is_draft attribute in the database to true, allowing the class or event to be published.

Notes

Acceptance Criteria

  • allows modification and saving class/event drafts without publishing
  • allows publishing class/event drafts, which are then included in the regular views
  • resolves bug without introducing new unexpected behavior
  • matches hi-fi (you can ignore coloring but please fix alignment/position of components to match hifi)

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