Skip to content

Conversation

@trollfot
Copy link
Contributor

This is a PR encompassing a few BS5 update for fields (classes) and a few upgrade of versions (tinyMCE, datepicker).
The PR is presented early to allow comments and change in direction.

@trollfot
Copy link
Contributor Author

Addressing : #499

@trollfot
Copy link
Contributor Author

Addressing: #452

@trollfot
Copy link
Contributor Author

Addressing : #431

@trollfot
Copy link
Contributor Author

Addressing: #448

@stevepiercy
Copy link
Member

@trollfot
Copy link
Contributor Author

I'm currently trying to touch a bit of everything in a "do it first then fix it" kind of way. I'll take care of the tests once we're happy with the fixes. This PR is by no means complete, it allows us to possibly communicate of the different issues. Sorry for the noise, if it produces any.

@stevepiercy stevepiercy marked this pull request as draft November 22, 2023 21:57
@stevepiercy
Copy link
Member

Ah, OK, in that case, I converted this PR to Draft status, which implies, "Please don't review yet, I'm working on it." When ready, you can either @ me or request a review from me and others, and change the status to ready for review.

@trollfot
Copy link
Contributor Author

The upload widget is flat out not working, because of the javascript behind it. This needs some heavier work.
In the meantime, checked inputs/password were altered to use an input group. this is a try out, and probably a matter of taste. The previous rendering was very unflattering, so...

@trollfot
Copy link
Contributor Author

Starting working on the functional tests. They need a bit of love since the introduction of the html5 "required" changes a lot of the validation tests.

@stevepiercy
Copy link
Member

@miohtama @mcdonc @ericof do any of you have any objection to adding both @delijati and @sbrunner as maintainers of Deform?

I'll give it another week, after which I'll add them as maintainers. Please let me know. Thank you!

@stevepiercy
Copy link
Member

I got a thumbs up from @mcdonc, and no other responses, so I've invited @delijati and @sbrunner to the Pylons organization Deform team.

Once they've both accepted, and pinged me back here, then I'll add them as maintainers on PyPI. @delijati and @sbrunner would you please verify your PyPI usernames so I can add you there as well?

There will also be some housekeeping chores to do.

Thank you for your patience and stepping up to maintain this package.

@sbrunner
Copy link

Thanks @trollfot, my pypi user name is the same: sbrunner :-)

@stevepiercy
Copy link
Member

@sbrunner PyPI invitation sent.

@delijati
Copy link
Contributor

same pypi name is also delijati

@stevepiercy
Copy link
Member

@delijati:

Error User 'delijati' does not have a verified primary email address and cannot be added as a Maintainer for project.

😢

Please verify your primary email and ping me back. Thank you!

@delijati
Copy link
Contributor

delijati commented Sep 17, 2025

@delijati:

Error User 'delijati' does not have a verified primary email address and cannot be added as a Maintainer for project.

😢

Please verify your primary email and ping me back. Thank you!

@stevepiercy Done sorry

@stevepiercy
Copy link
Member

@delijati invitation sent. Also if you haven't yet done so, please enable 2FA. There are some mean people out there.

@delijati
Copy link
Contributor

Now that the paperwork's done, @trollfot , could you fix that pull request so we can get it merged? seems to be only linting

@sbrunner
Copy link

@delijati you have write access on the GitHub repository?
I still have no write access to this repository.

@stevepiercy
Copy link
Member

stevepiercy commented Sep 17, 2025

@sbrunner the team has write access. You're a member of the team, and should inherit that access. Can you provide more information, including what you tried and observed?

@stevepiercy
Copy link
Member

Screenshot 2025-09-17 at 3 50 37 AM

@trollfot
Copy link
Contributor Author

Hello. I fixed the tests. But there is a build issue on MacOS that prevents the CI to finish

@sbrunner
Copy link

Sorry, @stevepiercy, I didn't look correctly; it's OK :-)

@sbrunner
Copy link

@trollfot the Python 3.9 ends in one month, I think that we can completely remove it.

@trollfot
Copy link
Contributor Author

MacOS still fails

@sbrunner
Copy link

Some of macos pass :-) => add

          - os: "macos-latest"
            py: "3.10" 

in exclude!

@sbrunner
Copy link

It still has an issue in MoneyInputWidgetTests!

Remove python badges : wrong versions
Updated python versions
@delijati
Copy link
Contributor

Does anyone have objections to merging this? I updated http://5.75.157.57/ so everybody can have a look

@trollfot
Copy link
Contributor Author

I've been ready for 2 years 👍

@delijati delijati merged commit 7d1b9ed into Pylons:main Sep 18, 2025
24 checks passed
@sbrunner
Copy link

Do we create a version 3.0.0-rc1?

@delijati
Copy link
Contributor

Yes. But for the final release we should cleanup the issues, docs, README a bit there are links that are outdated and so.

@delijati
Copy link
Contributor

@trollfot can you fix that branch so we can merge this too Pylons/deformdemo#129

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

5 participants