Move the Add-ons page from Advanced Topics to Resources - #206
Merged
Conversation
Every other page in Advanced Topics teaches something you do to your own app - downport it, run it off-stack, extend it, wire it to something. Add-ons teaches nothing of the kind: it is a list of what other people published, which is the same kind of page as References and Who Uses abap2UI5?, and it closes with the same invitation to add your own. The page moves to docs/resources/addons.md and the sidebar entry moves with it, next to Who Uses abap2UI5?. The eight inbound links across the site are rewritten to /resources/addons; nothing else on the page changes. The lead comment on Advanced Topics claimed the grouped section is seven entries and it was eight - with Add-ons gone it is seven, so the comment is now true rather than stale. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01A34xhKX8xmhxJTNx3cxsL4
There was a problem hiding this comment.
🟢 Approval recommended
The move is consistently reflected in sidebar/navigation and internal links, with no remaining references to the old /advanced/addons path.
Pull request overview
This PR reorganizes the VitePress documentation by moving the Add-ons page from Advanced Topics to Resources, aligning it with other ecosystem/reference-style pages and updating all internal links accordingly.
Changes:
- Moved the Add-ons page to
docs/resources/addons.mdand added it to the Resources sidebar section. - Removed the Add-ons entry from the Advanced Topics sidebar section and updated the surrounding explanatory comments.
- Rewrote all existing inbound links across the docs from
/advanced/addons(and../../advanced/addons.md) to/resources/addons(and../../resources/addons.md).
File summaries
| File | Description |
|---|---|
| docs/resources/addons.md | Adds the Add-ons page at its new location under Resources. |
| docs/.vitepress/config.mjs | Moves the sidebar entry from Advanced Topics to Resources and updates the related section comments. |
| docs/resources/deprecations.md | Updates internal link target to the new Add-ons URL. |
| docs/get_started/next.md | Updates internal link target to the new Add-ons URL. |
| docs/get_started/about.md | Updates internal link target to the new Add-ons URL. |
| docs/advanced/renaming.md | Updates internal link target to the new Add-ons URL. |
| docs/cookbook/device_capabilities/spreadsheet.md | Updates internal link target to the new Add-ons URL. |
| docs/cookbook/device_capabilities/barcode_scanning.md | Updates internal link target to the new Add-ons URL. |
| docs/cookbook/expert_more/lock.md | Updates relative link target to the new Add-ons page path. |
| docs/cookbook/expert_more/value_help.md | Updates relative link target to the new Add-ons page path. |
Review details
- Files reviewed: 9/10 changed files
- Comments generated: 0
- Review effort level: Lite
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Every other page in Advanced Topics teaches something you do to your own app — downport it, run it off-stack, extend it, wire it to something. Add-ons teaches nothing of the kind: it is a list of what other people published, which is the same kind of page as References and Who Uses abap2UI5?, and it closes with the same invitation to add your own project.
What changed
docs/advanced/addons.md→docs/resources/addons.md. The page content itself is untouched.docs/.vitepress/config.mjs: the entry moves out of Advanced Topics and into Resources, directly above Who Uses abap2UI5?. Both comment blocks say why, in the place the next reader will look./resources/addons:get_started/next.md,get_started/about.md,resources/deprecations.md,advanced/renaming.md,cookbook/device_capabilities/{spreadsheet,barcode_scanning}.md,cookbook/expert_more/{lock,value_help}.md.The lead comment on Advanced Topics claimed the grouped section is seven entries and it was eight — with Add-ons gone it is seven, so the comment is now true rather than stale.
Not in here
The old URL
/docs/advanced/addons.htmlbecomes a 404; this repository has no redirect mechanism and none is added here.Verification
npm run check— all gates green: build with no dead link and no orphan page (154 pages, unchanged),check:version,check:examples,check:conventions,check:playground,check:api-names,check:api-reference, unit tests.check:samplesskipped locally for want of a samples checkout, as documented; CI runs it with one, and the moved page carries nosamples:frontmatter.🤖 Generated with Claude Code
https://claude.ai/code/session_01A34xhKX8xmhxJTNx3cxsL4
Generated by Claude Code