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

feat: add editor component #3

Merged
merged 29 commits into from
Aug 7, 2024
Merged

feat: add editor component #3

merged 29 commits into from
Aug 7, 2024

Conversation

brian-baugher
Copy link
Contributor

@brian-baugher brian-baugher commented Jul 23, 2024

Major refactor to pull everything needed for the editor out of VE starter and into this VE lib. Adds export of Editor and the useMessage hooks. Allows usage of Editor with solely puckConfig, templateMetaData, and document as props. Corresponding VE Starter refactor PR: YextSolutions/pages-visual-editor-starter#108

Get everything ready for development (scripts, packages, directories, etc)
Major refactor to pull everything needed for the editor out of VE starter and into this VE lib.
Adds export of Editor and the useMessage hooks. Allows usage of Editor with solely puckConfig, templateMetaData, and document as props.
Corresponding VE Starter refactor PR coming shortly.

Untested.
package.json Outdated Show resolved Hide resolved
package.json Outdated Show resolved Hide resolved
src/hooks/index.ts Outdated Show resolved Hide resolved
src/internal/components/EntityField.tsx Outdated Show resolved Hide resolved
src/internal/components/internalEditor.tsx Outdated Show resolved Hide resolved
@brian-baugher brian-baugher marked this pull request as ready for review July 24, 2024 20:47
Base automatically changed from initialSetUp to main July 24, 2024 21:32
@brian-baugher brian-baugher marked this pull request as draft July 26, 2024 14:40
pnpm-lock.yaml Outdated
Comment on lines 4163 to 4164
[email protected]:
resolution: {integrity: sha512-4hLB8Py4zZce5s4yd9XzopqwVv/yGNhV1Bl8NTmCq1763HeK2+EwVTv+leGeL13Dnh2wfbqowVPXCIO0z4taYw==}

Choose a reason for hiding this comment

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

Legal Risk:
pako 1.0.11 was released under the Zlib license, a license currently prohibited by your organization. Merging is blocked until this is resolved

Recommendation:
Reach out to your security team or Semgrep admin to address this issue. In special cases, exceptions may be made for dependencies with violating licenses, however, the general recommendation is to avoid using a dependency under such a license

@brian-baugher brian-baugher marked this pull request as ready for review August 6, 2024 17:13
@mkilpatrick mkilpatrick merged commit 546b9a4 into main Aug 7, 2024
8 of 10 checks passed
@mkilpatrick mkilpatrick deleted the editorComponent branch August 7, 2024 21:22
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