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

Make Heroku API deployments conditional #1805

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

rhysyngsun
Copy link
Contributor

What are the relevant tickets?

N/A

Description (What does it do?)

We're having issues getting backend API deployments to heroku to work and I suspect the github action for that isn't behaving correct. I'm adding some conditionals here to only deploy the code if an environment variable is set to true

How can this be tested?

We'll have to test this by running a release candidate.

@rhysyngsun rhysyngsun added the Needs Review An open Pull Request that is ready for review label Nov 7, 2024
Copy link
Contributor

@shanbady shanbady left a comment

Choose a reason for hiding this comment

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

not sure how to test locally but changes look ok.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Needs Review An open Pull Request that is ready for review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants