-
-
Notifications
You must be signed in to change notification settings - Fork 248
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
Replace EditorEvents with Hookstate #5348
Conversation
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.
Everything looks good. At one place I like to draw your attention. Let me know your thought on that
@NPatel10 @HexaField 2 issues left in this PR which i will be working at night.
|
One more thing; I added |
@speigg Can you please point out with an example if such changes needs to be done in my PR. |
@zulqarnainhanif In EditorContainer.ts, you added useEffect(() => {
if (editorError) {
onEditorError(editorError.value)
}
}, [editorError]) This should be |
Summary
Checklist
npm run check
npm run lint
npm run test:packages
npm run build-client
References
#4340
QA Steps
git checkout pr_branch_name
npm install
npm run dev-reinit
npm run dev
List any additional steps required to QA the changes of this PR, as well as any supplemental images or videos.
Reviewers
@HexaField