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

Corriger la suggestion de déploiement Slack #223

Merged
merged 1 commit into from
Feb 9, 2023

Conversation

octo-topi
Copy link
Contributor

@octo-topi octo-topi commented Feb 9, 2023

🦄 Problème

Il existe une action Slack Deploy last version of an app.

Elle déploie:

Pour rappel, dans les autres actions Slack:

  • Publier = MER;
  • Déployer = MEP.

Le message est ambigü: Deploy = MEP alors qu'on peut effectuer cette action en dehors d'une application de production.

De plus, il est est en anglais alors que le reste est en français.
Du coup, les PO n'osent peut-être pas l'utiliser.

🤖 Proposition

Le renommer en Déployer sur cette application la dernière release publiée
On garde "release publiée" = release en recette.
Et "déployer sur cette application" mentionne le déploiement Scalingo, pas le déploiement = MEP.

💯 Pour tester

Relire

@octo-topi octo-topi added team-captains This is your captain speaking 👀 Tech Review Needed labels Feb 9, 2023
@octo-topi octo-topi self-assigned this Feb 9, 2023
@pix-bot-github
Copy link

Une fois l'application déployée, elle sera accessible à cette adresse https://bot-pr223.review.pix.fr
Les variables d'environnement seront accessibles sur scalingo https://dashboard.scalingo.com/apps/osc-fr1/pix-bot-review-pr223/environment

@octo-topi octo-topi force-pushed the tech-fix-slack-command-hint branch from 7a3b133 to 0a52765 Compare February 9, 2023 14:56
@github-actions github-actions bot merged commit bfab729 into main Feb 9, 2023
@github-actions github-actions bot deleted the tech-fix-slack-command-hint branch February 9, 2023 15:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants