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

fix: cannot emit load event in react wrapper #2107

Merged
merged 1 commit into from
Dec 21, 2021
Merged

Conversation

js87zz
Copy link
Contributor

@js87zz js87zz commented Dec 10, 2021

Please check if the PR fulfills these requirements

  • It's the right issue type on the title
  • When resolving a specific issue, it's referenced in the PR's title (e.g. fix #xxx[,#xxx], where "xxx" is the issue number)
  • The commit message follows our guidelines
  • Tests for the changes have been added (for bug fixes/features)
  • Docs have been added/updated (for bug fixes/features)
  • It does not introduce a breaking change or has a description of the breaking change

Description


Thank you for your contribution to TOAST UI product. 🎉 😘 ✨

@js87zz js87zz force-pushed the fix/react-wrapper-event branch from 23d8811 to fabcaae Compare December 20, 2021 08:18
Copy link
Contributor

@lja1018 lja1018 left a comment

Choose a reason for hiding this comment

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

리뷰 완료합니다.

Copy link

@adhrinae adhrinae left a comment

Choose a reason for hiding this comment

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

리뷰 완료합니다.

@@ -1,6 +1,5 @@
import React from 'react';
// @ts-ignore
import Viewer from '@toast-ui/editor/dist/toastui-editor-viewer';
import Viewer, { EventMap } from '@toast-ui/editor/dist/toastui-editor-viewer';

Choose a reason for hiding this comment

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

그냥 궁금한건데, Viewerdist 폴더에서 가져와야 하는 이유가 있나요?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

어플리케이션의 기본 export에서는 에디터만 제공합니다~! 뷰어는 번들 용량 문제도 있어서 별도 dist로 제공하고 있습니다.

@js87zz js87zz merged commit c63fe9a into master Dec 21, 2021
@js87zz js87zz deleted the fix/react-wrapper-event branch December 21, 2021 04:48
Copy link
Contributor

@jwlee1108 jwlee1108 left a comment

Choose a reason for hiding this comment

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

리뷰 완료합니다.

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.

4 participants