Skip to content

feat(notif-builder): Preview for slack#101164

Merged
leeandher merged 4 commits into
masterfrom
leander/preview-slack
Oct 9, 2025
Merged

feat(notif-builder): Preview for slack#101164
leeandher merged 4 commits into
masterfrom
leander/preview-slack

Conversation

@leeandher

Copy link
Copy Markdown
Member
image

Turns out the blockkit builder is a privledged link, so in case devs dont have a slack app to test with, there is now still a simple preview

@leeandher leeandher requested a review from a team October 8, 2025 16:18
@github-actions github-actions Bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Oct 8, 2025
cursor[bot]

This comment was marked as outdated.

Comment thread static/app/debug/notifications/previews/discordPreview.tsx Outdated
@codecov

codecov Bot commented Oct 8, 2025

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

Additional details and impacted files
@@           Coverage Diff            @@
##           master   #101164   +/-   ##
========================================
  Coverage   81.05%    81.05%           
========================================
  Files        8676      8676           
  Lines      384918    384912    -6     
  Branches    24319     24319           
========================================
- Hits       311977    311976    -1     
+ Misses      72582     72577    -5     
  Partials      359       359           

cursor[bot]

This comment was marked as outdated.

@Christinarlong Christinarlong 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.

do you really need a bot to access that link :bruh: . I'm p sure I was able to access w/o a bot or maybe you need certain perms 🫠

@leeandher

Copy link
Copy Markdown
Member Author

do you really need a bot to access that link :bruh: . I'm p sure I was able to access w/o a bot or maybe you need certain perms 🫠

Hmm, I think I misunderstood -- You can access block-kit builder without an app, but you do need access to a slack workspace and to be signed in in the browser. So in theory any sentry employee should be ok as long as they sign in to the sentry slack workspace in their browser, but I guess this can save them a click. Either way a nice to have! I wanna add it for MSTeams as well just for dev ex purposes

@leeandher leeandher force-pushed the leander/preview-slack branch from 3ea47c5 to fa306b8 Compare October 9, 2025 15:50
cursor[bot]

This comment was marked as outdated.

@leeandher leeandher enabled auto-merge (squash) October 9, 2025 16:28
@leeandher leeandher merged commit 9f8b59f into master Oct 9, 2025
45 checks passed
@leeandher leeandher deleted the leander/preview-slack branch October 9, 2025 16:32
@github-actions github-actions Bot locked and limited conversation to collaborators Oct 25, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants