-
Notifications
You must be signed in to change notification settings - Fork 165
Fix Altair GraphQL loading #1202
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Due to recent changes in Altair GraphQL v8.4.0 (updating the angular version and build system which uses native ESM module imports), the current static boilerplate no longer works properly. Updating the script tags as appropriate to conform to the new setup.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As shown in his code sandbox example (see Comment in altair-graphql/altair#2924 (comment) for details), these changes should solve the issue.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks alright, but shouldn't the runtime script be changed to type module too instead of just plain deleting it?
|
@CaiusSacu there is no more runtime.js |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #1202 +/- ##
===========================================
- Coverage 90.88% 68.26% -22.63%
===========================================
Files 50 123 +73
Lines 2425 4528 +2103
Branches 432 548 +116
===========================================
+ Hits 2204 3091 +887
- Misses 175 1339 +1164
- Partials 46 98 +52 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Due to recent changes in Altair GraphQL v8.4.0 (updating the angular version and build system which uses native ESM module imports), the current static boilerplate no longer works properly (Sorry about the breaking change!)
Updating the script tags as appropriate to conform to the new setup.
Closes #1201
Given this was a change in a recent version of Altair and the older versions still work with the older template, it might be prudent to use a fixed version of Altair to avoid breaking changes in newer versions breaking GraphQL.Server.Ui.Altair
Here's a codesandbox that uses the updated boilerplate (demo)
FYI jsdelivr allows you to set a major, minor or patch version, so it could be fixed to
cdn.jsdelivr.net/npm/altair-static@8/build/dist/orcdn.jsdelivr.net/npm/[email protected]/build/dist/to still get minor or patch version updates