Muka-is-home / api

0 stars 0 forks source link

Deployment test - 2 #23

Closed TrinityChristiana closed 1 year ago

TrinityChristiana commented 1 year ago

This PR prepares the API for deployment.

  1. Each PR will have a review app deployed.
  2. After merging a PR into main, Heroku will release it to the Staging URL: https://muka-staging-488c9e1c3e57.herokuapp.com/
  3. If any environmental variables need updating, modify them through the Heroku UI.

QA Steps:

Justin-Ferwerda commented 1 year ago

@TrinityChristiana When I test the two endpoints, the emails send but the response times out after about a minute so I don't get { "message": "Email Sent!" } in the response.

TrinityChristiana commented 1 year ago

@TrinityChristiana When I test the two endpoints, the emails send but the response times out after about a minute so I don't get `{

"message": "Email Sent!"

}` in the response.

Ahh!! It's because we're not adding the Email request to a queue. It's taking too long for the Heroku Dyno to resolve. Maybe we should test this when we get more endpoints.

drteresavasquez commented 1 year ago

@TrinityChristiana I am getting the following error when running the server: ModuleNotFoundError: No module named 'environ'

drteresavasquez commented 1 year ago

@TrinityChristiana I am getting the following error when running the server: ModuleNotFoundError: No module named 'environ'

I ran the install directly. I am not sure it was in the pip file and I used F5 to run the server and it seemed to work. I may need to pair on this to test it