Repository navigation
fix(docs): update broken message attachments links to current developer docs URL - #9162
Conversation
…er docs URL The message attachments documentation moved from docs.mattermost.com/developer/ to developers.mattermost.com. Two pages (bulk-export-tool, bulk-loading-data) still pointed at the old URL, which now returns 404. Update both to the live URL used elsewhere in the docs.
|
Hello @bloodsportshooting, Thanks for your pull request! A Core Committer will review your pull request soon. For code contributions, you can learn more about the review process here. Per the Mattermost Contribution Guide, we need to add you to the list of approved contributors for the Mattermost project. Please help complete the Mattermost contribution license agreement? This is a standard procedure for many open source projects. Please let us know if you have any questions. We are very happy to have you join our growing community! If you're not yet a member, please consider joining our Contributors community channel to meet other contributors and discuss new opportunities with the core team. |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (2)
Included review availability: Your plan provides up to 2 included reviews per hour; 1 remains after this review. 📝 WalkthroughWalkthroughThe changes replace deprecated Mattermost documentation links with the current Developers message attachments documentation link in two administration guides. ChangesDocumentation link updates
Estimated code review effort: 1 (Trivial) | ~2 minutes Merge Risk: ⚪ Minimal · up to This change only replaces two broken documentation links with the current message attachments reference; no actionable merge-blocking risk remains after normal checks and review. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
Full details: Docstring CoverageExplanation No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (2 skipped: 2 unsupported.) ✨ Finishing Touches 💡 1🛠️ Fix failing CI checks 💡
🧪 Generate unit tests (beta)
Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
|
/check-cla |
2 similar comments
|
/check-cla |
|
/check-cla |
|
@amyblais - Would you be able to help take care of the dev review for this one as it's just a minor link fix? |
amyblais
left a comment
amyblais
left a comment
There was a problem hiding this comment.
LGTM. Both occurrences of the dead docs.mattermost.com/developer/message-attachments.html link are replaced with developers.mattermost.com/integrate/reference/message-attachments/, which is the URL already used elsewhere in these docs. Thanks for the fix!
Generated by Claude Code
amyblais
added
3: Reviews Complete
@bloodsportshooting I accidentally merged here - a kind note that documentation updates are now done via the main mattermost repo. Would you be open to re-submitting this PR there? cc @Combs7th |
4 participants
Summary
Two documentation pages linked to
https://docs.mattermost.com/developer/message-attachments.html, which returns 404 — the message attachments reference now lives on the developer portal athttps://developers.mattermost.com/integrate/reference/message-attachments/(the URL already used elsewhere in these docs, e.g.source/end-user-guide/access/client-availability.rstandsource/integrations-guide/faq.rst).Changes
source/administration-guide/manage/bulk-export-tool.rst: update the message attachments link in the Post objectpropsfield description.source/administration-guide/onboard/bulk-loading-data.rst: same fix for the bulk-loading Post object table.Verification
https://docs.mattermost.com/developer/message-attachments.htmlreturns HTTP 404.