-
Notifications
You must be signed in to change notification settings - Fork 420
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
feat: notify AUS studio users of new studio versions #6893
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
1 Ignored Deployment
|
25c7957
to
26d1ba7
Compare
No changes to documentation |
Component Testing Report Updated Jun 18, 2024 3:10 PM (UTC)
|
26d1ba7
to
d831893
Compare
d831893
to
5b46348
Compare
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.
Overall looks good have some thoughts and questions
|
||
const MODULES_URL_VERSION = 'v1' | ||
|
||
const MODULES_URL = `https://sanity-cdn.com/${MODULES_URL_VERSION}/modules/` |
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.
Could we somehow share this with logic in CLI here? https://github.com/sanity-io/sanity/blob/next/packages/sanity/src/_internal/cli/util/getAutoUpdatesImportMap.ts#L11
Maybe we just need to lift this out in some constant?
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.
Hmm, yeah. Do we have precedent for sharing something like this between CLI and studio? Otherwise, happy to copy.
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.
I am not sure if there is, might be worth asking internally but for now okay with copy pasting as well
packages/sanity/src/core/studio/packageVersionStatus/PackageVersionStatusProvider.tsx
Show resolved
Hide resolved
packages/sanity/src/core/studio/packageVersionStatus/PackageVersionStatusProvider.tsx
Outdated
Show resolved
Hide resolved
Co-authored-by: Rune Botten <rbotten@gmail.com>
Co-authored-by: Rune Botten <rbotten@gmail.com>
Co-authored-by: Rune Botten <rbotten@gmail.com>
Description
As we move to certain packages being able to auto-update, we want end-users of the studio to know when to refresh their browsers and work with the most up-to-date packages.
This PR adds a Provider that will:
What to review
Unfortunately I think the reviewer ought to pull this down and review manually -- see below for why (because of limited ability to test).
Testing
I've added a few e2e tests. I'm not testing if the reload button actually reloads the page; I couldn't find an easy way for Playwright to listen to that event.