[XL] Move DUP from Terraform to SAM

bcgov / parks-reso-api

BC Parks Reservation System API

Apache License 2.0

13 stars 13 forks source link

Open JLWade opened 9 months ago

JLWade commented 9 months ago

Description Goal is to be able to deploy DUP (admin, public, and api) via SAM without the use of any terraform.

Acceptance Criteria

[ ] Full smoke test to make sure everything ported over properly
- [ ] DUP Public
- [ ] DUP Admin
- [ ] DUP API

Dev checklist

[ ] lookup how we do this in Data Register
[ ] Everything that is AWS side (ie. database, Cloudfront distributions, logs, S3,etc) will need to be setup in SAM
[ ] Implement DynamoDB layers
[ ] Write github actions so you can deploy it automatically when you do a change

Dianadec commented 6 months ago

@Christopher-walsh22 to add notes about the progress of this card at the time of iceboxing it.

Dianadec commented 4 months ago

holding off on merging this work until the hold pass timer is done.

Christopher-walsh22 commented 2 months ago

In the process of doing this SAM transition the steps identified for completion were as follows:

-Converting layers to SAM -Converting Lambdas to SAM -Converting Tests to SAM -Removing Terraform

All of this has been completed for DUP now. Instead of creating more tickets it seemed best to just leave a comment on this one.

manuji commented 1 week ago

The version on site footer shows "gitHubHash" instead of the version number.
Login to DUP-Admin using the BCService card does not work -> Error when trying enter the pairing code.
Booking confirmation emails are not received from test environment.

Nice to have features:

Do not display the option to cancel a pass to the non sysAdmin user or update the error messages with more details.