bcgov / parks-reso-public

Day Use Pass Public
Apache License 2.0
4 stars 17 forks source link

DUP email confirmation shows incorrect reservation date #471

Closed manuji closed 1 week ago

manuji commented 3 weeks ago

Scenario:

Expected:

Issue:

Christopher-walsh22 commented 1 week ago

@manuji @LindsayMacfarlane

I have been digging into this issue and just wanted to highlight some of what is happening. I don't think this is something that we need to action, but might be good to explore during any reworks of how we want to manage passes in the future but in order to address this we would need to fundamentally change parts of the DUP.

This issue has been created by having the AM opening hour set after the PM pass start time. The opening hour can be set, but the PM passes start at 1pm. While testing ticket #424 I believe Manuji changed the AM opening hour to 5pm. This makes sense for testing purposes, but I cant see any reason why this would be mirrored in the PROD environment as it would go against the booking instructions written on the website, and also just create an oxymoron of what type of pass a user would require. If we need to safeguard against an admin accidentally doing this we could set limitations on the admin side, but this has never been a problem before.

Please let me know how you want to proceed, I can either change the opening hour back to 7am so Manuji can complete her testing, or we can discuss a best path forward.

An additional note: The pass created in Manujis example was actually stored in the DB for the 30th, so the email was sending the correct date. It was the logic from the API which would not allow the AM pass to be created after the PM start time.

LindsayMacfarlane commented 1 week ago

Thank you @Christopher-walsh22 and @manuji -

Based on your input above Chris, could we change the opening hour back to 7am for Manuji to complete her testing?

I'd also like to hear how we should best track some of these issues as we start to scope bringing DUP into the new reservation system.

cc @Dianadec ^flag on the last sentence ;)

Christopher-walsh22 commented 1 week ago

Okay, I have changed the opening hours of all Gariabaldi facilities back to 7am, I will move this ticket over to the QA column, and if @manuji runs into this issue again we can reopen this for any further investigation.

manuji commented 1 week ago

Tested on TEST: Passed

FYI @LindsayMacfarlane