Skip to content

chore(slack): Added New commands to /sentry help#74017

Merged
iamrajjoshi merged 3 commits into
masterfrom
raj/mes-ref/bot-7
Jul 10, 2024
Merged

chore(slack): Added New commands to /sentry help#74017
iamrajjoshi merged 3 commits into
masterfrom
raj/mes-ref/bot-7

Conversation

@iamrajjoshi

@iamrajjoshi iamrajjoshi commented Jul 9, 2024

Copy link
Copy Markdown
Collaborator

After we added bot commands for support and docs, we should add them to the list of commands that we show the user when they ask for help via /sentry help

image

Addresses bot-7

@iamrajjoshi iamrajjoshi requested review from a team and vivianyentran July 9, 2024 16:51
@iamrajjoshi iamrajjoshi self-assigned this Jul 9, 2024
@iamrajjoshi iamrajjoshi requested a review from a team as a code owner July 9, 2024 16:51
@github-actions github-actions Bot added the Scope: Backend Automatically applied to PRs that change backend components label Jul 9, 2024

@cathteng cathteng left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

we should reorganize the text, some of the commands can be run in both a channel and a DM, only the link/unlink team commands can only be run in a channel

@codecov

codecov Bot commented Jul 9, 2024

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 78.15%. Comparing base (02217e1) to head (eadd493).
Report is 48 commits behind head on master.

Additional details and impacted files
@@             Coverage Diff             @@
##           master   #74017       +/-   ##
===========================================
+ Coverage   54.30%   78.15%   +23.84%     
===========================================
  Files        5598     6651     +1053     
  Lines      236100   297366    +61266     
  Branches    40459    51207    +10748     
===========================================
+ Hits       128206   232393   +104187     
+ Misses     106485    58705    -47780     
- Partials     1409     6268     +4859     
Files Coverage Δ
.../sentry/integrations/slack/message_builder/help.py 85.41% <100.00%> (+21.78%) ⬆️

... and 2314 files with indirect coverage changes

Comment thread src/sentry/integrations/slack/message_builder/help.py Outdated
Comment thread src/sentry/integrations/slack/message_builder/help.py Outdated
Co-authored-by: vivianyentran <20403606+vivianyentran@users.noreply.github.com>
@iamrajjoshi iamrajjoshi merged commit 9c61faa into master Jul 10, 2024
@iamrajjoshi iamrajjoshi deleted the raj/mes-ref/bot-7 branch July 10, 2024 19:10
@github-actions github-actions Bot locked and limited conversation to collaborators Jul 26, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Scope: Backend Automatically applied to PRs that change backend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants