Skip to content

feat: generate index pages for api/ and api/{framework}/#592

Merged
akudev merged 1 commit into
mainfrom
fix/api-index-pages
Jun 15, 2026
Merged

feat: generate index pages for api/ and api/{framework}/#592
akudev merged 1 commit into
mainfrom
fix/api-index-pages

Conversation

@akudev

@akudev akudev commented Jun 15, 2026

Copy link
Copy Markdown
Member

After doc generation, scan the existing version directories and build index.html pages listing all available versions. This makes /api/, /api/openui5/, and /api/sapui5/ navigable instead of 404.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Adds generation of navigable index.html pages under the published API docs site so /api/, /api/openui5/, and /api/sapui5/ don’t 404 and instead list available generated versions.

Changes:

  • Updates the workflow_dispatch input description to clarify version entry formats.
  • Adds a gh-pages step that scans generated version directories and writes api/index.html plus per-framework api/{framework}/index.html pages.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/generate-api-docs.yml
Comment thread .github/workflows/generate-api-docs.yml
@akudev akudev force-pushed the fix/api-index-pages branch from 16093f1 to d2164d9 Compare June 15, 2026 11:25

@codeworrior codeworrior left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

some minor things, but looks okay

Comment thread .github/workflows/generate-api-docs.yml
Comment thread .github/workflows/generate-api-docs.yml
Comment thread .github/workflows/generate-api-docs.yml Outdated
Comment thread .github/workflows/generate-api-docs.yml Outdated
After doc generation, scan the existing version directories
and build index.html pages listing all available versions.
This makes /api/, /api/openui5/, and /api/sapui5/ navigable
instead of 404.
@akudev akudev force-pushed the fix/api-index-pages branch from d2164d9 to 305cdfc Compare June 15, 2026 11:41
Comment thread .github/workflows/generate-api-docs.yml
<title>UI5 TypeScript API Reference</title>
<meta name="description" content="Crawlable TypeScript API documentation for OpenUI5 and SAPUI5">
<meta name="viewport" content="width=device-width,initial-scale=1">
<link rel="canonical" href="https://ui5.github.io/typescript/api/">

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

${BASE}

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Claude says: The main index uses << 'EOF' (single-quoted — no variable expansion) so the canonical URL is hardcoded. ${BASE} is used in the framework-specific index which uses << EOF (unquoted — expansion works). The hardcoded value is correct since this URL never changes.

@codeworrior codeworrior Jun 15, 2026

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Then, Claude has no interest in a consistent coding style, I fear. It's okay for AI, but not for me.

"since this URL never changes" is true for all places where ${BASE} is used.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated no new comments.

@akudev akudev merged commit 01b4752 into main Jun 15, 2026
8 checks passed
@akudev akudev deleted the fix/api-index-pages branch June 15, 2026 12:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants