Skip to content
This repository has been archived by the owner on Feb 22, 2023. It is now read-only.

Ping person who kicked off deployment in Slack notification #2052

Closed
2 tasks
krysal opened this issue Dec 15, 2022 · 0 comments · Fixed by #2059
Closed
2 tasks

Ping person who kicked off deployment in Slack notification #2052

krysal opened this issue Dec 15, 2022 · 0 comments · Fixed by #2059
Assignees
@krysal
Labels
🤖 aspect: dx Concerns developers' experience with the codebase 🧰 goal: internal improvement Improvement that benefits maintainers, not users 🟩 priority: low Low priority and doesn't need to be rushed 🔒 staff only Restricted to staff members

Comments

@krysal
Copy link
Member

krysal commented Dec 15, 2022

Quoting @sarayourfriend:

Problem

Currently people could merge and then not know when their staged change is ready for testing in staging or easily forget about it.

Description

Let's ping the person who merged or kicked off the deployment in the Slack notification so that they can more easily keep track of it without having to watch a specific Slack channel.

Basically, we have to update the following files here:

  • .github/actions/staging-frontend-deploy/action.yml
  • .github/actions/production-frontend-deploy/action.yml

Additional context

Related to WordPress/openverse-infrastructure#154 by @sarayourfriend
@krysal krysal added 🚦 status: awaiting triage Has not been triaged & therefore, not ready for work ✨ goal: improvement Improvement to an existing user-facing feature labels Dec 15, 2022
@krysal krysal added 🟩 priority: low Low priority and doesn't need to be rushed 💻 aspect: code Concerns the software code in the repository 🔒 staff only Restricted to staff members 🧰 goal: internal improvement Improvement that benefits maintainers, not users 🤖 aspect: dx Concerns developers' experience with the codebase and removed 🚦 status: awaiting triage Has not been triaged & therefore, not ready for work ✨ goal: improvement Improvement to an existing user-facing feature 💻 aspect: code Concerns the software code in the repository labels Dec 15, 2022
@krysal krysal self-assigned this Dec 15, 2022
This was referenced Dec 15, 2022
Merged
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@krysal krysal

Labels
🤖 aspect: dx Concerns developers' experience with the codebase 🧰 goal: internal improvement Improvement that benefits maintainers, not users 🟩 priority: low Low priority and doesn't need to be rushed 🔒 staff only Restricted to staff members
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Ping deployer on production Slack notification WordPress/openverse-frontend
1 participant
@krysal