-
Notifications
You must be signed in to change notification settings - Fork 15
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
Salvage exising ideas #1
Comments
Hi Thomas, First, that repo is just a fork of @mike-marcacci's original repo: https://github.com/mike-marcacci/angular-schema-form-builder , so it is mostly their ideas. And to add more to this thread, @rowino's existing builder is at: There is aIso a demo of here; The code looks nice and is vanilla ES6, which is good as we want people from all frameworks all to be able to contribute to it. |
One thing I feel is important with the builder is that you should be able to use it as part of one's own solutions, for example I want to make it possible for my users to define their own structures. |
yes, absolutely! this is exactly what I need it for :D <div ng-controller="FormBuilderController">
<sbpalette></sbpalette>
<form sb-schema="schema" sb-form="form" sb-model="model"></form>
<sbinspector></sbinspector>
</div> |
Something along those lines, yes. I would guess that is what @joelwkent is looking for as well. |
WRT your example, I'd like to mention that I would like to keep the implementation framework-agnostic and ES 6. Not sure how it should be implemented smoothly to for example work nicely both as an angular directive and react-component. |
So I guess it should be based on https://github.com//json-schema-form/json-schema-form-core, right? Not really sure right now how one would architect this, either. You would have to write a small wrapper for both, right? How does (or will) ASF do this? But you're right, this should certainly be the goal. |
Forget about the vanilla ES6 stuff I just said. How embarrassing. So making a pure ES6 version of it is obviously not doable. |
Is the core already at a stage where it makes sense to develop the builder on top of it? Does ASF is it as well? |
Actually, I do not think we should build on that until 1.0 is released. So if I could make a suggestion, it would be to now focus on what we want to do with it and try and break down that to a road map.
* There are some security issues with that, which should be mentioned in the docs. Users can, if it isn't sanitized, insert scripts into the descriptions. |
I agree. I guess the first step would be to extract the important stuff from the existing app and making that into an easy to include angular module? / directive? Since the code is already vanilla js I dont think we need a grunt task, right? The one included for the test app throws a lot if errors for me anyway. |
Grunt? Isn't it gulp? |
Of course, no idea why I keep mixing up those two... |
Well..no, I don't think so, we will need to add testing and stuff like that, and that is usually handled by the build tools. |
hm, I guess you're right. I'm fairly new to this whole process as you can probably tell ;-) So what would be the next step to extract a reusable library from this app? |
Yeah, basically, I suppose. |
so, I finally managed to get everything into a working directive. There's still a lot of code-cleanup, bugfixing and restructuring to do though, so it'll still take a bit until I'm confident enough to create a pull-request. |
I'm gonna try to get some life in here over from json-schema-form/angular-schema-form/issues/304 ;-)
I guess the first step would be to collect the existing ideas from all the different places and merge them here into a roadmap, right?
I really like @nicklasb's ideas for the GUI. Having a drag-and-drop interface is probably the easiest way to deal with more complicated things like arrays.
https://github.com/nicklasb/angular-schema-form-builder
Also: would it make sense to create a Gitter for this repo or should we stay in https://gitter.im/json-schema-form/angular-schema-form-projects ?
Thoughts?
The text was updated successfully, but these errors were encountered: