Skip to content

Moving to BS5 with a bit of 3rd party updating - #531

Merged
delijati merged 42 commits into
Pylons:mainfrom
novareto:main
Sep 18, 2025
Merged

Moving to BS5 with a bit of 3rd party updating#531
delijati merged 42 commits into
Pylons:mainfrom
novareto:main

Conversation

@trollfot

Copy link
Copy Markdown
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
Copy Markdown
Contributor Author

Addressing : #499

@trollfot

Copy link
Copy Markdown
Contributor Author

Addressing: #452

@trollfot

Copy link
Copy Markdown
Contributor Author

Addressing : #431

@trollfot

Copy link
Copy Markdown
Contributor Author

Addressing: #448

@stevepiercy

Copy link
Copy Markdown
Member

@trollfot

Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
Contributor

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

@stevepiercy

Copy link
Copy Markdown
Member

@sbrunner PyPI invitation sent.

@delijati

Copy link
Copy Markdown
Contributor

same pypi name is also delijati

@stevepiercy

Copy link
Copy Markdown
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

delijati commented Sep 17, 2025

Copy link
Copy Markdown
Contributor

@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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
Contributor

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

@stevepiercy

stevepiercy commented Sep 17, 2025

Copy link
Copy Markdown
Member

@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
Copy Markdown
Member
Screenshot 2025-09-17 at 3 50 37 AM

@trollfot

Copy link
Copy Markdown
Contributor Author

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

@sbrunner

Copy link
Copy Markdown
Contributor

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

@sbrunner

Copy link
Copy Markdown
Contributor

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

@trollfot

Copy link
Copy Markdown
Contributor Author

MacOS still fails

@sbrunner

Copy link
Copy Markdown
Contributor

Some of macos pass :-) => add

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

in exclude!

@sbrunner

Copy link
Copy Markdown
Contributor

It still has an issue in MoneyInputWidgetTests!

Remove python badges : wrong versions
Updated python versions
@delijati

Copy link
Copy Markdown
Contributor

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

@trollfot

Copy link
Copy Markdown
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
Copy Markdown
Contributor

Do we create a version 3.0.0-rc1?

@delijati

Copy link
Copy Markdown
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
Copy Markdown
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