-
Notifications
You must be signed in to change notification settings - Fork 90
[BD-5145] Add testing tabs and redirects #11439
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
Conversation
|
🤖 Automated Reviewer Assignment: I have automatically added reviewers based on the following:
|
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.
Pull request overview
This pull request adds missing "Testing" redirect pages for various messaging channels (WhatsApp, Webhooks, SMS/MMS/RCS, Push, LINE, Email) and reorganizes the testing tabs on the main testing page to be in alphabetical order. The PR also fixes a broken link in the testing landing page.
Changes:
- Created redirect pages for WhatsApp, Webhooks, SMS/MMS/RCS, Push, LINE, and Email that redirect users to the central testing documentation with the appropriate tab parameter
- Updated existing redirect pages for in-app messages and content cards to include tab parameters
- Reorganized tabs in the main sending_test_messages.md file to be in alphabetical order (Banners, Content Card, Email, In-app message, LINE, Push, SMS/MMS and RCS, Webhook, WhatsApp)
- Adjusted page_order values across multiple files to accommodate the new testing pages in their respective channel sections
- Fixed the link and title in testing_and_more.md from "Send Test Messages" to "Sending Test Messages" with the correct URL
Reviewed changes
Copilot reviewed 19 out of 20 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
_docs/_user_guide/message_building_by_channel/whatsapp/testing.md |
New redirect page for WhatsApp testing with page_order 10 |
_docs/_user_guide/message_building_by_channel/whatsapp/meta_resources.md |
Updated page_order from 11 to 12 to accommodate new testing page |
_docs/_user_guide/message_building_by_channel/whatsapp/faqs.md |
Updated page_order from 10 to 11 to accommodate new testing page |
_docs/_user_guide/message_building_by_channel/webhooks/testing.md |
New redirect page for Webhook testing with page_order 11 |
_docs/_user_guide/message_building_by_channel/sms_mms_rcs/testing.md |
New redirect page for SMS/MMS/RCS testing with page_order 13 |
_docs/_user_guide/message_building_by_channel/sms_mms_rcs/sms.md |
Updated page_order from 13 to 14 to accommodate new testing page |
_docs/_user_guide/message_building_by_channel/sms_mms_rcs/rcs.md |
Updated page_order from 14 to 15 to accommodate new testing page |
_docs/_user_guide/message_building_by_channel/sms_mms_rcs/mms.md |
Updated page_order from 15 to 16 to accommodate new testing page |
_docs/_user_guide/message_building_by_channel/push/troubleshooting.md |
Updated page_order from 23 to 24 to accommodate new testing page |
_docs/_user_guide/message_building_by_channel/push/testing.md |
New redirect page for Push testing with page_order 23 |
_docs/_user_guide/message_building_by_channel/push/faq.md |
Updated page_order from 80 to 25 to properly sequence with other pages |
_docs/_user_guide/message_building_by_channel/line/testing.md |
New redirect page for LINE testing with page_order 5 |
_docs/_user_guide/message_building_by_channel/in-app_messages/testing.md |
Updated redirect URL to include tab parameter |
_docs/_user_guide/message_building_by_channel/email/testing.md |
New redirect page for Email testing with page_order 13 |
_docs/_user_guide/message_building_by_channel/email/faq.md |
Updated page_order from 14 to 15 to accommodate new testing page |
_docs/_user_guide/message_building_by_channel/email/best_practices.md |
Updated page_order from 13 to 14 to accommodate new testing page |
_docs/_user_guide/message_building_by_channel/content_cards/testing.md |
Updated redirect URL to include tab parameter |
_docs/_user_guide/engagement_tools/campaigns/testing_and_more/sending_test_messages.md |
Reorganized tabs alphabetically and moved content between tabs |
_docs/_user_guide/engagement_tools/campaigns/testing_and_more.md |
Fixed link title and URL for "Sending Test Messages" |
lydia-xie
left a comment
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.
lgtm!
Why are you making this change? (required)
Added missing messaging channels to the testing tabs and added missing "Testing" redirect headers to the respective messaging channel sections.
Related PRs, issues, or features (optional)
BD-5145
Feature release date (optional)
Contributor checklist
Submitting for review
If your PR meets the above requirements, select Ready for review, then add a reviewer:
@braze-inc/docs-teamin a comment below.braze-inc/docs-teaminstead.Thanks for contributing! We look forward to reading your work.