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

fixed react warnings #65

Closed

Conversation

janstuemmel
Copy link

  • fixed React PropTypes warning
  • fixed React creatClass warning

related with #60 #61

* fixed React PropTypes warning
* fixed React creatClass warning
janstuemmel pushed a commit to janstuemmel/react-mce that referenced this pull request Jun 13, 2017
* added karma/mocha testsuite
* deleted old test


related to instructure-react#65
Jan Stümmel added 2 commits June 14, 2017 02:46
* added gulp build and example buid
* edited readme 
* edited package.json for npm release
@janstuemmel
Copy link
Author

I forked the repo, added some tests and published it on npm at react-mce

@ben-burlingham
Copy link

ben-burlingham commented Jun 27, 2017

Thank you for this! Any idea when it will make it into master?

edit last commits a year ago...doesn't seem likely 😢

@janstuemmel
Copy link
Author

janstuemmel commented Jun 28, 2017

it won't. my fork implements some useful testcases, but a few more are welcome.
just use npm install --save react-mce

indeed my PR is not ment to be merged, because there to many changes and commits

@gugadev
Copy link

gugadev commented Aug 6, 2017

@mzabriskie dude, can you look at this? seems to work :)

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