Skip to content
This repository has been archived by the owner on Mar 1, 2024. It is now read-only.

Stateless form #14

Open
joshgagnon opened this issue Jan 27, 2016 · 2 comments
Open

Stateless form #14

joshgagnon opened this issue Jan 27, 2016 · 2 comments

Comments

@joshgagnon
Copy link

Because my project useless redux, I've using a stateless fork of react-json-editor, here. Validation is done in the reducer (and later I will put normalisation there too).
If you are interested in PR I will write some tests for it, but if so, do you think it would make more sense to just create a new form component (formStateless or something) rather than the ugly branching?

@ismaelga
Copy link
Owner

ismaelga commented Feb 1, 2016

This sounds interesting to me. But I won't accept a PR of it right now. I have a lot of things to introduce in react-json-editor. And having 2 different ways of using it (stateful/stateless) would make it even harder to make sure everything is working.

Thanks!

@bherila
Copy link

bherila commented Sep 5, 2017

Also looks interesting... any update?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants