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

Add Heroku config #16

Merged
merged 1 commit into from
Nov 20, 2023
Merged

Add Heroku config #16

merged 1 commit into from
Nov 20, 2023

Conversation

jazairi
Copy link
Contributor

@jazairi jazairi commented Nov 20, 2023

Why these changes are being introduced:

We need a Procfile and app.json for our Heroku config.

Relevant ticket(s):

https://mitlibraries.atlassian.net/browse/ENGX-239

How this addresses that need:

This adds a Procfile and app.json with the initial settings we'll
need.

Side effects of this change:

None.

@mitlib mitlib temporarily deployed to tacos-api-pipeline-pr-16 November 20, 2023 17:07 Inactive
@jazairi jazairi temporarily deployed to tacos-api-pipeline-pr-16 November 20, 2023 18:08 Inactive
@jazairi jazairi changed the title Add Procfile Add Procfile and app.json Nov 20, 2023
@jazairi jazairi changed the title Add Procfile and app.json Add Heroku config Nov 20, 2023
@JPrevost
Copy link
Member

@jazairi would you mind adding a commit (or opening a ticket) to change prod db to be pg? This still uses sqlite on Heroku. I have no preference as to whether that's a new commit or a new ticket.

The rest of this is 👍🏻 so if you prefer the new ticket approach I'll approve.

@JPrevost JPrevost self-assigned this Nov 20, 2023
@jazairi jazairi temporarily deployed to tacos-api-pipeline-pr-16 November 20, 2023 18:55 Inactive
@jazairi jazairi temporarily deployed to tacos-api-pipeline-pr-16 November 20, 2023 18:57 Inactive
@JPrevost
Copy link
Member

Closing and re-opening the PR to test the build process

@JPrevost JPrevost closed this Nov 20, 2023
@JPrevost JPrevost reopened this Nov 20, 2023
@mitlib mitlib temporarily deployed to tacos-api-pipeline-pr-16 November 20, 2023 20:31 Inactive
Copy link
Member

@JPrevost JPrevost left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'll open new tickets to get GraphiQL working (it's dev only right now) as well as creating a splash page.

:shipit:

Why these changes are being introduced:

We need a Procfile and app.json for our Heroku config.

Relevant ticket(s):

https://mitlibraries.atlassian.net/browse/ENGX-239

How this addresses that need:

This adds a Procfile and app.json with the initial settings we'll
need.

Side effects of this change:

The pg has been added to production.
@jazairi jazairi temporarily deployed to tacos-api-pipeline-pr-16 November 20, 2023 22:05 Inactive
@jazairi jazairi merged commit 172794c into main Nov 20, 2023
2 checks passed
@jazairi jazairi deleted the engx-239-tacos-pipeline branch November 20, 2023 22:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants