jsonform / jsonform

Build forms from JSON Schema. Easily template-able. Compatible with Bootstrap 3 out of the box.
https://jsonform.github.io/jsonform/playground/index.html
MIT License
2.72k stars 553 forks source link

[meta] Maintenance going forward #412

Closed tchapi closed 1 year ago

tchapi commented 1 year ago

:wave: Hi all contributors,

Unfortunately lately I haven't had the bandwidth to look at issues / pull requests and my day-to-day job is taking too much time. Also, I have little interest to put hours into this library as I'm not using it anymore.

@sdetweil, would you be interested in taking ownership of the maintenance of this library ? I've created an organization so it's easy to add you to it with the correct merge rights.

Thanks ๐Ÿ™๐Ÿผ

sdetweil commented 1 year ago

@tchapi yes, I would be willing

is the doc in the same repo?

who manges/pays for/supports the jsonform playground website, and how do updates/examples get added/removed there

tchapi commented 1 year ago

is the doc in the same repo?

Everything is in this repo on Github, there's nothing else.

who manages/pays for/supports the jsonform playground website

This is Github Pages, directly from the master branch of this repo. It's free. There is no maintenance or hosting involved.

how do updates/examples get added/removed there

It's in the playground folder

sdetweil commented 1 year ago

ok, thanks

sdetweil commented 1 year ago

i see I cannot set labels on issues..

tchapi commented 1 year ago

Should be good now

sdetweil commented 1 year ago

I have a wiki update for #411 , and I made a separate branch.. how can I expose it for any review?

seems it master or nothing.. I will post that to the issue I suppose

tchapi commented 1 year ago

Yes, the wiki is not versionned (as in git) but it's revisioned (here) so you're safe

sdetweil commented 1 year ago

thx.. posted in original issue.. will wait a little

sdetweil commented 1 year ago

I think we should merge PRs 414 and 422

at what point do u decide to refresh npmjs lib? and how? I don't have any libs there

tchapi commented 1 year ago

I merged #422.

For #414, I think it would be better to update the whole optional dependency ...

sdetweil commented 1 year ago

sorry don't understand the dependency thing,

got it now, didn't look where the file came from..

shouldnt all these be in a package,json dependencies list.. I don't know where to get them, and a post install script could move them/link them where they need to be.

sdetweil commented 1 year ago

jsv.js from github project (also 10 yrs old) isn't as big as the one here.. (by half)

tchapi commented 1 year ago

I'm all in favour of putting it in package.json, but jsonform doesn't use any package dependency manager yet (like yarn or npm). This would be a good thing to add but it's not straightforward I think

sdetweil commented 1 year ago

can u guide me where the components come from/should come from. I see the jQuery stuff

jsv seems soooo different and has dependencies

tchapi commented 1 year ago

jsv is here: https://github.com/garycourt/JSV/commits/master The version we use is 0aa11852537069b0830569ef1eab11a36b65b3ab - you can pin the dependency to this specific commit if ever it's not possible to update (I doubt so)

sdetweil commented 1 year ago

thx. I also have trouble w the jQuery plugins. every time I select download I get taken to jQuery download, which doesn't provide these separated out in the downloaded zip

tchapi commented 1 year ago

Can you find them on npm? I'd be easier

sdetweil commented 1 year ago

didn't look there!

sdetweil commented 1 year ago

did we want to try for bootstrap 5?

sdetweil commented 1 year ago

how does test work?

tchapi commented 1 year ago

did we want to try for bootstrap 5?

Yes sure (see https://github.com/jsonform/jsonform/issues/364 for instance) but I have no time on my hands to do so

how does test work?

It's only old visual tests in https://github.com/jsonform/jsonform/blob/master/tests/index.html โ€”ย they're not really useful anymore I suppose

sdetweil commented 1 year ago

can u write me two or three paragraphs on what u think the work is, and I can start. not asking u to do it.

my email is my userid at gmail

sdetweil commented 1 year ago

ah, i see.. I Have never produced a production web site before so haven't investigated using any kind of framework, classes, styles, etc.. and i didn't pay attention to the reasons why there were specific classes on specific elements

so in some cases we can replace the classes with the new (form-group), and test for impacts and in some cases classes/plugins went away.. (dropdown) from the bootstrap core (v5)

tchapi commented 1 year ago

and in some cases classes/plugins went away.. (dropdown) from the bootstrap core (v5)

It's included, you just have to include popper.js with it: see https://getbootstrap.com/docs/5.0/components/dropdowns/

sdetweil commented 1 year ago

ok, i think the strategy should be migrate to V4, then to V5.

stale[bot] commented 1 year ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

Rndwiga commented 10 months ago

Was the update to V5 ever done?

sdetweil commented 10 months ago

no.

Rndwiga commented 10 months ago

That's sad. Before I waste time hacking my way with the upgrades, and do a PR, is there a fork that has the upgrades?

sdetweil commented 10 months ago

not that I am aware of.. I was working on it, but never got done.. now swamped with other stuff

Rndwiga commented 10 months ago

Oh okay. Did you have an opportunity to look at what was suggested on the fork on issue #401 ? I could start by upgrading using those suggestions

sdetweil commented 10 months ago

I saw them, but did not evaluate them at the time I was starting to update ..

some look awesome, some I'm not sure what they mean..

so net, no

Rndwiga commented 10 months ago

I am going to jump Bs4 and go straight to Bs5. Since that's where I need to use the project on. Can we do that on a branch that I can be pushing PRs to?

sdetweil commented 10 months ago

yes, use the upgrades branch, just created

Rndwiga commented 10 months ago

Thanks!

Rndwiga commented 10 months ago

Well, it seems nothing is broken yet apart from the UI looking ugly but functional. What do you feel about the following:

https://rndwiga.github.io/jsonform/playground/index.html

sdetweil commented 10 months ago

seems reasonable..