Skip to content
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

Update to support ^1.0.0-rc.1 #341

Merged
merged 20 commits into from
Aug 22, 2023
Merged

Conversation

jwartofsky-yext
Copy link
Contributor

Update to support ^1.0.0-rc.0

J=SLAP-2892
TEST=auto,manual

Auto tests pass.

Can run studio with npm run dev using
^1.0.0-rc.0 after modification.

Note that this change requires this update in PagesJS, so will either need to update PagesJS locally or wait until the next PagesJS release.

@jwartofsky-yext
Copy link
Contributor Author

This should not be merged until the next version of PagesJS is released to npm. This PR will also need to be updated to use the new PagesJS version.

@jwartofsky-yext jwartofsky-yext force-pushed the dev/support-yext-pages-1.0.0-rc.0 branch from fcbd0c5 to 4b960af Compare August 18, 2023 17:45
J=SLAP-2892
TEST=auto,manual

Auto tests pass.

Can run studio with npm run dev using
^1.0.0-rc.0 after modification.

yext/pages@4cf67c9

Note that this change requires this update in PagesJS, so will
either need to update locally or wait until the next PagesJS
release.
Retested with npm run dev

Studio appears and works as expected
@jwartofsky-yext jwartofsky-yext force-pushed the dev/support-yext-pages-1.0.0-rc.0 branch from 1716a06 to fbfce13 Compare August 21, 2023 18:05
@jwartofsky-yext jwartofsky-yext changed the title Update to support ^1.0.0-rc.0 Update to support ^1.0.0-rc.1 Aug 21, 2023
@jwartofsky-yext
Copy link
Contributor Author

Tested with dyext/pages-1.0.0-rc.1 and npm run dev. All is working as expected. This should be good to go.

@oshi97 oshi97 force-pushed the dev/support-yext-pages-1.0.0-rc.0 branch from 61bc105 to 5a3822a Compare August 22, 2023 17:24
package.json Outdated Show resolved Hide resolved
package.json Outdated Show resolved Hide resolved
packages/studio/package.json Outdated Show resolved Hide resolved
@jwartofsky-yext jwartofsky-yext requested a review from nmanu1 August 22, 2023 19:53
@jwartofsky-yext jwartofsky-yext merged commit a7f536e into main Aug 22, 2023
@@ -33,7 +33,7 @@
"@babel/preset-env": "^7.22.4",
"@babel/preset-react": "^7.22.3",
"@babel/preset-typescript": "^7.21.5",
"@yext/pages": "^1.0.0-beta.14",
"@yext/pages": "^1.0.0-rc.1",
Copy link
Contributor

Choose a reason for hiding this comment

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

I think the third party notices for studio-plugin need to be updated to reflect this change

@oshi97 oshi97 deleted the dev/support-yext-pages-1.0.0-rc.0 branch August 23, 2023 14:15
tmeyer2115 added a commit that referenced this pull request Aug 28, 2023
## Features
- An Info button was added next to the Live Preview button. When
clicked, this displays the version number of Studio. (#346)
- Number-type Props now have a Field Picker. (#344)

## Changes
- The Live Preview button is now hidden when Studio is run in Storm. The
PagesJS Dev Server is also not started. (#348)
- Studio now supports PagesJS `^1.0.0-rc.1`. (#341)

## Bug Fixes
- Immer is now correctly installed as a direct dependency of Studio.
(#349)
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