Repository navigation
Make the manual developer-only; the case for decision makers moves out - #384
Merged
Merged
Conversation
abap2UI5 for decision makers is its own site (abap2UI5/test-docs, https://abap2ui5.github.io/test-docs/). What the manual carried for them goes there instead of standing twice: - docs/evaluate.md, resources/cost_calculator, resources/who_uses and resources/references are removed; redirect stubs keep their addresses (the Evaluate ones keep the #fragment - the new page has the same ids) - the cost calculator's code, styles and test, and the landing layout that only the Evaluate page used, go with them - the front door is the developer's: "See how it works" instead of "Evaluate", the security card links Security, the price card is gone, and the fourth tile hands the page for decision makers over - In a Nutshell, Hello World and the Quickstart link the new site - /test-docs/ is a neighbouring deployment for check:cross-site Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_0134Dawy9Fb26HLtDiHWfnT1
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.
abap2UI5 for decision makers is now a site of its own: abap2UI5/test-docs, published at https://abap2ui5.github.io/test-docs/. The manual carried the same content a second time. With this PR it stays there only, and the manual is for developers.
Companion PR: abap2UI5/test-docs#3. Merge that one first, so the redirects below land on sections that exist.
Changes
evaluate.md,resources/cost_calculator,resources/who_uses,resources/references.docs/public/keep the old addresses working, all withtarget="_self". Every stub now points at the new site:evaluate.htmlandget_started/evaluate.htmlkeep the#fragment. The framework README links#what-you-keepand#when-to-use-it-and-when-not, and the new page keeps every section id.resources/cost_calculator.html→#what-it-costsresources/who_uses.htmlandresources/references.html→#who-runs-ittheme/cost-calculator.js,cost-model.js, its hook intheme/index.jsandsite-js/site.js, its CSS andtest/cost-calculator.test.mjs), and thelandinglayout, which only the Evaluate page used (build-site.mjs,docs.css).check:cross-site:/test-docs/is added as a neighbouring deployment.Kept on purpose: the blog essays #2 and #33–#35 (RAP / freestyle / low-code or abap2UI5). They are technical pieces with code and architecture for developers and do not repeat the decision-maker page.
Test
Passed locally:
npm test: 279 passnpm run build(169 pages, no dead internal links)docs:buildcheck:cross-site,check:design,check:images,check:line-length,check:vocabulary,check:playground,check:api-names,check:conventionsNot fully verified here:
check:versioncould not reach the release API, so it compared only the three places against each other.check:samples,check:examplesandcheck:api-referencewere not run; CI covers them.Generated by Claude Code