Skip to content

[1/9] [Constants]: Add API migration audit script - #1045

Closed
daviddamilola wants to merge 2 commits into
mainfrom
api-migration-audit
Closed

daviddamilola wants to merge 2 commits into
mainfrom
api-migration-audit

Conversation

@daviddamilola

Copy link
Copy Markdown
Contributor

Fetches the old and new GraphQL schemas, runs codegen for both, diffs them, validates every document of a page against both APIs and reports the schema types used by the page that changed. Output goes to the git-ignored apps/platform/migrations folder.

Usage: yarn workspace @ot/constants apiMigrationAudit --page

[scoped_application]: Short description

PR Tittle Format: [Genetics || Platform || PackageName || AppConfig]: Short description mentioning the affected page and/or section component

Description

Please include a summary of the change and which issue is fixed. List any dependencies that are required for this change.

Issue: (link)
Deploy preview: (link)

Type of change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce.

  • Test A
  • Test B

Checklist:

  • I have commented my code, particularly in hard-to-understand areas
  • My changes generate no new warnings
  • I have made corresponding changes to the documentation

Fetches the old and new GraphQL schemas, runs codegen for both, diffs them,
validates every document of a page against both APIs and reports the schema
types used by the page that changed. Output goes to the git-ignored
apps/platform/migrations folder.

Usage: yarn workspace @ot/constants apiMigrationAudit --page <name>
@daviddamilola daviddamilola reopened this Oct 20, 2026
@daviddamilola
daviddamilola deleted the api-migration-audit branch October 20, 2026 11:34
@daviddamilola
daviddamilola restored the api-migration-audit branch October 20, 2026 12:35
@daviddamilola daviddamilola reopened this Oct 20, 2026
@daviddamilola daviddamilola changed the title [Constants]: Add API migration audit script [1/9] [Constants]: Add API migration audit script Oct 20, 2026
@carcruz

carcruz commented Oct 21, 2026

Copy link
Copy Markdown
Member

Superseded by the re-cut new API stack (#1059–#1083). Commits and authorship are kept there. Tracking: opentargets/issues#4596. Closing.

@carcruz carcruz closed this Oct 21, 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