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

#162693956 Fix Heroku Configurations #23

Closed
wants to merge 1 commit into from

Conversation

dnuwa
Copy link
Contributor

@dnuwa dnuwa commented Dec 17, 2018

What does this PR do?

  • Fixes Heroku deployment errors.

Description of Task to be completed?

  • add configurations for app to read Heroku environment variables

How should this be manually tested?

What are the relevant pivotal tracker stories?

#162693956

@dnuwa dnuwa changed the title bg(Fix Heroku)Fix Heroku Configurations #162693956 Fix Heroku Configurations Dec 17, 2018
@brucemakallan brucemakallan requested a deployment to ah-frontend-thanos-stagi-pr-23 December 19, 2018 06:52 Abandoned
@brucemakallan brucemakallan temporarily deployed to ah-frontend-thanos-stagi-pr-23 December 19, 2018 06:52 Inactive
@dnuwa dnuwa force-pushed the bg-Heroku-Configurations-162693956 branch from 24a83ef to ba3ab96 Compare December 19, 2018 07:17
@dnuwa dnuwa had a problem deploying to ah-frontend-thanos-stagi-pr-23 December 19, 2018 07:17 Failure
@dnuwa dnuwa force-pushed the bg-Heroku-Configurations-162693956 branch from ba3ab96 to f5711ad Compare December 19, 2018 07:47
@dnuwa dnuwa temporarily deployed to ah-frontend-thanos-stagi-pr-23 December 19, 2018 07:47 Inactive
@dnuwa dnuwa force-pushed the bg-Heroku-Configurations-162693956 branch from f5711ad to f84cd1b Compare December 19, 2018 08:58
@dnuwa dnuwa temporarily deployed to ah-frontend-thanos-stagi-pr-23 December 19, 2018 08:58 Inactive
package.json Outdated
"start": "serve -s dist",
"build": "webpack --config webpack.prod.js",
"start:dev": "NODE_ENV=development env-cmd .env webpack-dev-server --open",
"start:prod": "yarn run build && serve dist",
Copy link
Contributor

Choose a reason for hiding this comment

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

why are you using yarn?

@dnuwa dnuwa force-pushed the bg-Heroku-Configurations-162693956 branch from f84cd1b to ef08c3b Compare December 19, 2018 09:32
@dnuwa dnuwa had a problem deploying to ah-frontend-thanos-stagi-pr-23 December 19, 2018 09:32 Failure
@dnuwa dnuwa force-pushed the bg-Heroku-Configurations-162693956 branch from ef08c3b to 3d941ee Compare December 19, 2018 09:42
@dnuwa dnuwa had a problem deploying to ah-frontend-thanos-stagi-pr-23 December 19, 2018 09:43 Failure
@dnuwa dnuwa force-pushed the bg-Heroku-Configurations-162693956 branch from 3d941ee to ecbec5f Compare December 19, 2018 09:50
@dnuwa dnuwa had a problem deploying to ah-frontend-thanos-stagi-pr-23 December 19, 2018 09:50 Failure
@dnuwa dnuwa force-pushed the bg-Heroku-Configurations-162693956 branch from ecbec5f to 53bb3fa Compare December 19, 2018 09:54
@dnuwa dnuwa had a problem deploying to ah-frontend-thanos-stagi-pr-23 December 19, 2018 09:54 Failure
@dnuwa dnuwa force-pushed the bg-Heroku-Configurations-162693956 branch from 53bb3fa to 255d735 Compare December 19, 2018 10:07
@dnuwa dnuwa temporarily deployed to ah-frontend-thanos-stagi-pr-23 December 19, 2018 10:08 Inactive
@dnuwa dnuwa force-pushed the bg-Heroku-Configurations-162693956 branch from 255d735 to d23dddd Compare December 19, 2018 12:30
@dnuwa dnuwa temporarily deployed to ah-frontend-thanos-stagi-pr-23 December 19, 2018 12:30 Inactive
@dnuwa dnuwa force-pushed the bg-Heroku-Configurations-162693956 branch from d23dddd to ee3979b Compare December 19, 2018 12:47
@dnuwa dnuwa temporarily deployed to ah-frontend-thanos-stagi-pr-23 December 19, 2018 12:47 Inactive
@dnuwa dnuwa temporarily deployed to ah-frontend-thanos-staging December 19, 2018 12:51 Inactive
@dnuwa dnuwa force-pushed the bg-Heroku-Configurations-162693956 branch from ee3979b to 59446b8 Compare December 19, 2018 13:26
@dnuwa dnuwa temporarily deployed to ah-frontend-thanos-stagi-pr-23 December 19, 2018 13:26 Inactive
@dnuwa dnuwa force-pushed the bg-Heroku-Configurations-162693956 branch from 59446b8 to 848e4dc Compare December 19, 2018 13:42
@dnuwa dnuwa temporarily deployed to ah-frontend-thanos-stagi-pr-23 December 19, 2018 13:42 Inactive
@dnuwa dnuwa force-pushed the bg-Heroku-Configurations-162693956 branch from 848e4dc to 015f427 Compare December 19, 2018 14:25
@dnuwa dnuwa had a problem deploying to ah-frontend-thanos-stagi-pr-23 December 19, 2018 14:25 Failure
@dnuwa dnuwa force-pushed the bg-Heroku-Configurations-162693956 branch from 015f427 to c986add Compare December 19, 2018 14:28
@dnuwa dnuwa had a problem deploying to ah-frontend-thanos-stagi-pr-23 December 19, 2018 14:28 Failure
@dnuwa dnuwa force-pushed the bg-Heroku-Configurations-162693956 branch from c986add to 5bf1640 Compare December 19, 2018 15:37
@dnuwa dnuwa had a problem deploying to ah-frontend-thanos-stagi-pr-23 December 19, 2018 15:37 Failure
@dnuwa dnuwa force-pushed the bg-Heroku-Configurations-162693956 branch from 5bf1640 to f09f0c5 Compare December 19, 2018 15:41
@dnuwa dnuwa temporarily deployed to ah-frontend-thanos-stagi-pr-23 December 19, 2018 15:41 Inactive
@dnuwa dnuwa force-pushed the bg-Heroku-Configurations-162693956 branch from f09f0c5 to 8660ec8 Compare December 19, 2018 15:43
- configures app to read heroku environment variables

[#162693956]
@dnuwa dnuwa force-pushed the bg-Heroku-Configurations-162693956 branch from 8660ec8 to 5f2e195 Compare December 19, 2018 16:19
@codeclimate
Copy link

codeclimate bot commented Dec 19, 2018

Code Climate has analyzed commit 5f2e195 and detected 0 issues on this pull request.

View more on Code Climate.

Copy link
Collaborator

@brucemakallan brucemakallan left a comment

Choose a reason for hiding this comment

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

Please work on coverage

@dnuwa dnuwa closed this Dec 19, 2018
@dnuwa dnuwa deleted the bg-Heroku-Configurations-162693956 branch December 19, 2018 18:21
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.

None yet

3 participants