Skip to content

Update JSON schema from openlibrary-client#12723

Merged
mekarpeles merged 1 commit into
internetarchive:masterfrom
hornc:master
May 12, 2026
Merged

Update JSON schema from openlibrary-client#12723
mekarpeles merged 1 commit into
internetarchive:masterfrom
hornc:master

Conversation

@hornc

@hornc hornc commented May 12, 2026

Copy link
Copy Markdown
Collaborator

incorporates from openlibrary-client:

Technical

Testing

Screenshot

Stakeholders

Copilot AI review requested due to automatic review settings May 12, 2026 04:40

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

Updates Open Library’s JSON schema documentation to match recent changes in openlibrary-client, correcting local $ref JSON Pointer syntax and removing a confusing/unused id field from the Work schema.

Changes:

  • Fix $ref fragments to use valid JSON Pointer syntax (#/...) in schema files.
  • Remove the undocumented id property from work.schema.json.

Reviewed changes

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

File Description
openlibrary/schemata/work.schema.json Fixes $ref to author_role and removes the confusing id property from the Work schema.
openlibrary/schemata/shared_definitions.json Fixes local $ref to author_key to use valid JSON Pointer syntax.

@mekarpeles
mekarpeles merged commit 3b1e1a9 into internetarchive:master May 12, 2026
9 checks passed
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