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

[FEATURE] Deployer pix-pro sur les reviews app (PIX-1263) #173

Merged
merged 1 commit into from
Sep 24, 2020

Conversation

bpetetot
Copy link
Contributor

@bpetetot bpetetot commented Sep 21, 2020

🦄 Problème

Nous avons besoin de déployer pix-pro sur les reviews app

🤖 Solution

  • Ajout de la variable d'environnement SITE sur les review app pix-site-review et pix-pro-review sur scalingo.

  • Modification du package.json pour avoir des commandes distinctes en fonction d'environnement et de l'application:

Par exemple:

npm run build # build l'application en fonction de la variable d'environnement SITE définie

npm run build:site  # build l'application pix-site (variable SITE pré-définie)

npm run build:pro  # build l'application pix-pro (variable SITE pré-définie)
  • Modification du template de pull request pour prendre en compte pix-pro (le pix-review-router n'a pas encore été modifié)

🌈 Remarques

RAS

✨ Review App

Pour tester.

Vous devez voir pix-site ici: https://site-pr173.review.pix.fr/
Vous devez voir pix-pro ici: https://pix-pro-review-pr173.osc-fr1.scalingo.io/

@bpetetot bpetetot changed the base branch from dev to pix-1260-build-pix-site-and-pix-pro September 21, 2020 13:57
README.md Outdated Show resolved Hide resolved
Base automatically changed from pix-1260-build-pix-site-and-pix-pro to dev September 22, 2020 13:27
@bpetetot bpetetot merged commit 8cce847 into dev Sep 24, 2020
@bpetetot bpetetot deleted the pix-1263-pix-pro-review-app branch September 24, 2020 13:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
👀 Tech Review Needed team-evaluation PR relatives à l'expérience d'évaluation team-prescription
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants