Skip to content

Update docs#818

Merged
AlexAxthelm merged 1 commit into
mainfrom
docs/schema
May 29, 2026
Merged

Update docs#818
AlexAxthelm merged 1 commit into
mainfrom
docs/schema

Conversation

@AlexAxthelm
Copy link
Copy Markdown
Collaborator

@AlexAxthelm AlexAxthelm commented May 28, 2026

followup to #816

Copilot AI review requested due to automatic review settings May 28, 2026 14:35
@github-actions
Copy link
Copy Markdown

Expected version change and release notes

🚨 WARNING: This PR is not expected to trigger a new version

To trigger a version bump, use at least one conventional commit message in this branch. See: https://www.conventionalcommits.org/en/v1.0.0/

@github-actions
Copy link
Copy Markdown

Azure Static Web Apps: Your stage site is ready! Visit it here: https://proud-glacier-0f640931e-818.westus2.2.azurestaticapps.net

Copy link
Copy Markdown
Contributor

Copilot AI left a 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 PR updates the generated HTML schema documentation for the pathway metadata and timeseries schemas under public/schema/, changing how referenced schemas (notably label.v1) are displayed within the embedded publication sections.

Changes:

  • Removed the inlined Label property documentation (e.g., full/short) from publication.title sections.
  • Renamed the “All of” tab/breadcrumb display text from Label to label.v1.json.
  • Introduced an inline “schema generation” failure message where a referenced schema could not be loaded.

Reviewed changes

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

File Description
public/schema/pathwayTimeseries.v1.html Updates generated schema docs for publication-related sections, but now shows unresolved $ref output and an inline generation error message.
public/schema/pathwayMetadata.v1.html Same generated-doc changes as timeseries; introduces missing/incorrect Label rendering and an inline generation error message.

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

Comment thread public/schema/pathwayMetadata.v1.html
Comment thread public/schema/pathwayMetadata.v1.html
Comment thread public/schema/pathwayMetadata.v1.html
Comment thread public/schema/pathwayMetadata.v1.html
Comment thread public/schema/pathwayTimeseries.v1.html
Comment thread public/schema/pathwayTimeseries.v1.html
Comment thread public/schema/pathwayTimeseries.v1.html
Comment thread public/schema/pathwayTimeseries.v1.html
@AlexAxthelm AlexAxthelm requested a review from jacobvjk May 29, 2026 06:15
Copy link
Copy Markdown
Collaborator

@jacobvjk jacobvjk left a comment

Choose a reason for hiding this comment

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

lgtm

@AlexAxthelm AlexAxthelm merged commit 4175e51 into main May 29, 2026
12 checks passed
@AlexAxthelm AlexAxthelm deleted the docs/schema branch May 29, 2026 08:52
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