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

Collaborate on changes? #8

Open
Coridyn opened this issue Mar 29, 2015 · 2 comments
Open

Collaborate on changes? #8

Coridyn opened this issue Mar 29, 2015 · 2 comments

Comments

@Coridyn
Copy link

Coridyn commented Mar 29, 2015

Hi ulion,

Thanks for maintaining this library, it's been a great help. I've been working with jsonform but keep running into problems with the original library.

I'd like to make some changes to add some features and fix some issues but some of them are breaking changes. Instead of creating a whole new fork to compete with yours, would you be interested in collaborating on these changes?

The main areas I want to improve are:

  1. Create standard array functions - make a set of standard functions that can be used to work with arrays
    • Remove duplicated code in 'array' and 'tabarray'
    • Make it easier for everyone else to create their own array views
  2. Add extra view callbacks (e.g. view.onAfterRender())
    • Let parent views manipulate/interact with the child template after it's been generated (no easy eay to do this at the moment).
  3. Bring tests up to date
  4. Split up source files
    • Working with a ~4500 line file gets really frustrating.

Would you be interested in discussing these items or accepting pull requests in these areas?

Thanks,
-Coridyn

@ulion
Copy link
Owner

ulion commented Mar 29, 2015

Hi, Coridyn, glad you can work on jsonform, I did some fix and little
enhancement on the original fork, and now maintain my dev fork, add feature
I will using, fix bug if I can.

so, I will help you, and as long as the code is relative stable and works,
I can merge your changes.

2015-03-29 16:01 GMT+08:00 Coridyn notifications@github.com:

Hi ulion,

Thanks for maintaining this library, it's been a great help. I've been
working with jsonform but keep running into problems with the original
library.

I'd like to make some changes to add some features and fix some issues but
some of them are breaking changes. Instead of creating a whole new fork to
compete with yours, would you be interested in collaborating on these
changes?

The main areas I want to improve are:

Create standard array functions - make a set of standard functions
that can be used to work with arrays
- Remove duplicated code in 'array' and 'tabarray'
- Make it easier for everyone else to create their own array views
2.

Add extra view callbacks (e.g. view.onAfterRender())
- Let parent views manipulate/interact with the child template after
it's been generated (no easy eay to do this at the moment).
3.

Bring tests up to date
4.

Split up source files
- Working with a ~4500 line file gets really frustrating.

Would you be interested in discussing these items or accepting pull
requests in these areas?

Thanks,
-Coridyn


Reply to this email directly or view it on GitHub
#8.

Ulion

@Coridyn
Copy link
Author

Coridyn commented Mar 29, 2015

Thanks, I'll start making changes on my fork and send through pull requests as I finish things.

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

No branches or pull requests

2 participants