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

Can book gigs that have already happened #13

Open
liamquinn14 opened this issue Nov 19, 2024 · 3 comments
Open

Can book gigs that have already happened #13

liamquinn14 opened this issue Nov 19, 2024 · 3 comments

Comments

@liamquinn14
Copy link
Collaborator

Version

v3

Severity

High

Suggested Priority

High

Observed Behaviour

Currently the dates for all of the gigs in our site are in the past - June 2024. Nonetheless, users are still able make bookings for all of them. This is problematic because users may be excited to attend a gig, book a ticket, then realise that they can't actually go because it has already taken place.

Expected Behaviour

There should not be an option to make a booking for gigs that have already taken place. Furthermore, there should be some visual indication that tells the user that the gig's date has already passed. Perhaps the date could be coloured red?

Reproduction Steps

  1. Follow setup instructions for the project
  2. Run project with python app.py
  3. Follow instructions in README.md to create a user
  4. Log in as the new user
  5. Navigate to /gigs
  6. Click on 'More details & booking' for a gig of your choice
  7. Buy a ticket
@PaulNGilson
Copy link
Contributor

Unless you can make a good case for this, I think this is going to be a wontfix - we're just building the site, the customer gets to maintain their list of (future) gigs.

@PaulNGilson PaulNGilson added the wontfix This will not be worked on label Nov 20, 2024
@PaulNGilson
Copy link
Contributor

I guess this issue could be split in two - better indications on gigs in the past, vs not being able to book them in the first place?

@liamquinn14
Copy link
Collaborator Author

Thanks for your feedback but I believe we have an obligation to the client to fulfil the requirements of the specification and the user stories they have provided us.

I have just reseeded the database to include a gig with a date in the future - in order to test whether "gigs in the past are displayed differently" (as per the spec) and whether gigs in the past are "clearly labelled" (as per the user stories).

In the current state of the application, past and future gigs are not displayed differently (see screenshot below).

Screenshot 2024-11-20 at 11 33 35

We simply cannot ship while the application doesn't meet the specified requirements. I will add this as a separate issue now.

I will concede that the specification doesn't include guidance on whether or not gigs in the past can be booked. However we should consult with Jo in order to update the specification, since being able to book gigs in the past would inevitably cause unnecessary problems to users.

@PaulNGilson PaulNGilson added p1-high sev2-high and removed wontfix This will not be worked on labels Nov 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants