Skip to content

feat(notification integration): 📝 notification integration doc #767

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

Draft
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

ziracmo
Copy link
Contributor

@ziracmo ziracmo commented Apr 15, 2025

📝 Notification Integration Documentation

@ziracmo ziracmo requested a review from AlteredCoder April 15, 2025 14:50
@ziracmo ziracmo self-assigned this Apr 15, 2025
@ziracmo ziracmo force-pushed the notification-integration-doc branch from 8803848 to 41f0764 Compare April 16, 2025 08:43
Copy link

This pull request is automatically being deployed by Amplify Hosting (learn more).

Access this pull request here: https://pr-767.d1to60jd2gb6y6.amplifyapp.com

@ziracmo ziracmo force-pushed the notification-integration-doc branch from 41f0764 to 894ecbc Compare April 16, 2025 08:51
@ziracmo ziracmo force-pushed the notification-integration-doc branch from 894ecbc to a472b25 Compare April 22, 2025 10:14
@ziracmo
Copy link
Contributor Author

ziracmo commented May 19, 2025

@AlteredCoder The Webhook documentation should also inform users about the ping that is not available in edition mode

---


## Link your workspace
Copy link
Contributor

Choose a reason for hiding this comment

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

I think we need to add simlple overview sentence at the very beginning to make the pl understand how it works overall and make them intuite what configuration and rules mean. (could be good in general overview too)
--- example but let's work on phrasing a bit ---
Adding an integration will Create an integration configuration specific to your slack workspace.
Then you can add rules to select what event trigger notifications to what slack channel.

---- in just 2 sentence we then clarify that integration is global to the workspace and user should NOT expect to select a channel straight away, also we mention "configuration" and "rules" thus explaining the other tabs of the section

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@rr404 What do you think of:

"Connecting an integration will create a configuration specific to your Slack workspace.
You can then define rules to control which events trigger notifications and which Slack channel they’re sent to."

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.

3 participants