-
-
Notifications
You must be signed in to change notification settings - Fork 229
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
🎉 share newly created DIs in Slack #4571
base: create-di-topic-tags
Are you sure you want to change the base?
Conversation
Quick links (staging server):
Login:
SVG tester:Number of differences (default views): 0 ✅ Edited: 2025-02-18 11:48:32 UTC |
55f09da
to
b8da097
Compare
468c34e
to
c1249cf
Compare
b8da097
to
7d78a3e
Compare
c1249cf
to
3d4609d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
sehr gut
|
||
const response = await fetch(url, fetchData) | ||
|
||
if (!response.ok) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think there's anything we can do to catch this, but the request was 200'ing for me, but not posting to slack when I put zalgo text in it.
I don't think that'll be something we have to worry about from our authors though 😅
Send a message to the
#data-insights-pitches
channel in Slack on DI creation.The Slack message includes the title, an optional note and the image (see screenshot).
The idea of having a
#data-insights-pitches
channel is that there is a very early editorial review that decides whether it makes sense to flesh out a data insight.For testing, set
SLACK_DI_PITCHES_CHANNEL_ID
to the ID of the#data-insights-pitches
channel (I just added you to the channel). The secret key forSLACK_BOT_OAUTH_TOKEN
is in 1password.The env var isn't set on staging yet, so it's currently only testable locally. I'll figure this out tomorrow, but I wanted to open up for review anyway.
Screenshot
This is part 9 of 9 in a stack made with GitButler: