Skip to content

Conversation

@AltamashShaikh
Copy link
Contributor

Mention Slack in README.md and faq,md

snake14
snake14 previously approved these changes Sep 23, 2025
Copy link
Contributor

@snake14 snake14 left a comment

Choose a reason for hiding this comment

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

Looks good 👍

I left a couple optional/nitpick comments.

Create Custom Alerts + Be notified by email/SMS/Slack!

Alerts are a great way to get notified of changes on your website. Want to know if your new product hits less than 100 sales in a week or your new article attracts more than 200 visitors a day? Create alerts that make sense to you. Be notified by email or SMS when the conditions for your alerts are met. Stay on top of your website!
Alerts are a great way to get notified of changes on your website. Want to know if your new product hits less than 100 sales in a week or your new article attracts more than 200 visitors a day? Create alerts that make sense to you. Be notified by email, SMS or Slack when the conditions for your alerts are met. Stay on top of your website!
Copy link
Contributor

@snake14 snake14 Sep 23, 2025

Choose a reason for hiding this comment

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

Nitpick: Many prefer the Oxford comma (comma before the conjunction at the end of a list).

Suggested change
Alerts are a great way to get notified of changes on your website. Want to know if your new product hits less than 100 sales in a week or your new article attracts more than 200 visitors a day? Create alerts that make sense to you. Be notified by email, SMS or Slack when the conditions for your alerts are met. Stay on top of your website!
Alerts are a great way to get notified of changes on your website. Want to know if your new product hits less than 100 sales in a week or your new article attracts more than 200 visitors a day? Create alerts that make sense to you. Be notified by email, SMS, or Slack when the conditions for your alerts are met. Stay on top of your website!

@AltamashShaikh AltamashShaikh merged commit 366f830 into 5.x-dev Sep 24, 2025
7 checks passed
@AltamashShaikh AltamashShaikh deleted the KB-1391-update-docs branch September 24, 2025 05:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants