-
Notifications
You must be signed in to change notification settings - Fork 270
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
[Editing Integration] Integrate /editing/render endpoint #1806
Conversation
packages/create-sitecore-jss/src/templates/nextjs/src/lib/next-config/plugins/cors-header.js
Show resolved
Hide resolved
@@ -83,7 +87,7 @@ export type GraphQLRequestClientConfig = { | |||
* @returns An instance of a GraphQL Request Client ready to send GraphQL requests. | |||
*/ | |||
export type GraphQLRequestClientFactory = ( | |||
config: Omit<GraphQLRequestClientConfig, 'apiKey'> | |||
config?: Omit<GraphQLRequestClientConfig, 'apiKey'> |
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.
NOTE: made config argument optional since initially it didn't match the interface we have here
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.
This is looking great! See some feedback.
...sitecore-jss/src/templates/nextjs-xmcloud/src/lib/page-props-factory/plugins/preview-mode.ts
Outdated
Show resolved
Hide resolved
...sitecore-jss/src/templates/nextjs-xmcloud/src/lib/page-props-factory/plugins/preview-mode.ts
Outdated
Show resolved
Hide resolved
packages/sitecore-jss-nextjs/src/editing/editing-data-service.ts
Outdated
Show resolved
Hide resolved
packages/sitecore-jss-nextjs/src/editing/editing-render-middleware.ts
Outdated
Show resolved
Hide resolved
packages/sitecore-jss-nextjs/src/editing/editing-render-middleware.ts
Outdated
Show resolved
Hide resolved
packages/sitecore-jss-nextjs/src/editing/editing-render-middleware.ts
Outdated
Show resolved
Hide resolved
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.
👍
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.
Just left one comment for now, I'll do another look around with a fresh mind later
Description / Motivation
[next.js]
Integrated a new Metadata Edit Mode in /api/editing/render endpoint.EditingMetadataPreviewData
type andisEditingMetadataPreviewData
guard function.GraphQLEditingService
class to fetch editing data in Metadata Edit Mode.[sitecore-jss]
Introduced /editing submodule that contains all editing related functionality. Editing utils are now available in /editing submodule. Editing utils exported from /utils marked as deprecated.GraphQLRequestClient
now can accept custom 'headers' in the constructor or via createClientFactoryTesting Details
Types of changes