Skip to content

[DO NOT MERGE] INTER-2472: Update OpenAPI schema URL - #45

Closed
erayaydin wants to merge 1 commit into
mainfrom
chore/openapi-repo-rename-inter-2472
Closed

erayaydin wants to merge 1 commit into
mainfrom
chore/openapi-repo-rename-inter-2472

Conversation

@erayaydin

Copy link
Copy Markdown
Member

Warning

Do not merge this PR until fingerprintjs/openapi#480 merged and the repository renamed.

Changes

  • package.json: generateTypes schema URL
  • contributing.md: repository link

GitHub does not redirect project GitHub Pages URLs after a rename, so
pnpm generateTypes 404s until the URL is updated.

Related-Task: INTER-2472
@erayaydin
erayaydin requested a balanced review from Copilot September 24, 2026 19:12
@erayaydin erayaydin self-assigned this Sep 24, 2026
@github-actions

Copy link
Copy Markdown

This PR will not create a new release 🚀

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot review overview

🔵 Needs a closer look

The required upstream PR #480 remains open and the repository rename has not occurred.

Review effort: Balanced
Findings: None

What changed in this PR

Updates OpenAPI references for the planned repository rename.

Changes:

  • Updates the schema-generation URL.
  • Updates the contributing guide repository link.
File Description
package.json Points type generation to the renamed schema host.
contributing.md Links to the renamed repository.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@erayaydin
erayaydin marked this pull request as ready for review September 25, 2026 09:00
@erayaydin

Copy link
Copy Markdown
Member Author

We decided to not update this repository yet

@erayaydin erayaydin closed this Sep 29, 2026
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.

2 participants