Skip to content

Add functionality for subteams#44

Merged
rickycodes merged 1 commit intomainfrom
fix-tag
May 26, 2023
Merged

Add functionality for subteams#44
rickycodes merged 1 commit intomainfrom
fix-tag

Conversation

@rickycodes
Copy link
Copy Markdown
Contributor

@rickycodes rickycodes commented May 26, 2023

Okay! our deployment today failed to tag the subteam because I didn't have this configured properly. I had assumed the functionality for tagging individuals and subteams was identical, but I was wrong! see: https://api.slack.com/reference/surfaces/formatting#mentioning-groups

I've tested this out in a private channel in our Slack instance and it works properly now:

image

@rickycodes rickycodes requested a review from a team as a code owner May 26, 2023 19:06
Copy link
Copy Markdown
Member

@Gudahtt Gudahtt left a comment

Choose a reason for hiding this comment

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

LGTM!

@rickycodes rickycodes requested review from a team and Gudahtt May 26, 2023 19:09
@rickycodes rickycodes merged commit db469fd into main May 26, 2023
@rickycodes rickycodes deleted the fix-tag branch May 26, 2023 19:11
rickycodes added a commit to MetaMask/KeyringController that referenced this pull request May 27, 2023
This fixes the broken subteam tagging that was patched in MetaMask/action-npm-publish#44

same as MetaMask/core#1398
rickycodes added a commit to MetaMask/metamask-module-template that referenced this pull request May 27, 2023
## Description

This fixes the broken subteam tagging that was patched in MetaMask/action-npm-publish#44

same as MetaMask/core#1398

same as MetaMask/KeyringController#232
rickycodes added a commit to MetaMask/KeyringController that referenced this pull request May 27, 2023
* Make use of the new subteam functionality

This fixes the broken subteam tagging that was patched in MetaMask/action-npm-publish#44

same as MetaMask/core#1398

* don't bump here
rickycodes added a commit to MetaMask/metamask-module-template that referenced this pull request May 27, 2023
## Description

This fixes the broken subteam tagging that was patched in MetaMask/action-npm-publish#44

same as MetaMask/core#1398

same as MetaMask/KeyringController#232
rickycodes added a commit that referenced this pull request May 27, 2023
Update to match functionality landed in #44
@rickycodes rickycodes mentioned this pull request May 27, 2023
rickycodes added a commit that referenced this pull request May 30, 2023
Update to match functionality landed in #44
@legobeat legobeat mentioned this pull request Sep 11, 2023
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.

2 participants