-
Notifications
You must be signed in to change notification settings - Fork 163
Moving to BS5 with a bit of 3rd party updating #531
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
Conversation
|
Addressing : #499 |
|
Addressing: #452 |
|
Addressing : #431 |
|
Addressing: #448 |
|
59 test failures: https://github.com/Pylons/deform/actions/runs/6956721991/job/18928147423?pr=531#step:7:6539. See https://github.com/Pylons/deform/blob/main/contributing.md#when-to-add-or-edit-functional-tests for how to fix the tests. |
|
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. |
…, regarding errors and required
…Same attempt wit the datetime input, where required and error were added
|
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 |
|
The upload widget is flat out not working, because of the javascript behind it. This needs some heavier work. |
|
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. |
|
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 PyPI invitation sent. |
|
same pypi name is also delijati |
😢 Please verify your primary email and ping me back. Thank you! |
@stevepiercy Done sorry |
|
@delijati invitation sent. Also if you haven't yet done so, please enable 2FA. There are some mean people out there. |
|
Now that the paperwork's done, @trollfot , could you fix that pull request so we can get it merged? seems to be only linting |
|
@delijati you have write access on the GitHub repository? |
|
@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? |
|
Hello. I fixed the tests. But there is a build issue on MacOS that prevents the CI to finish |
|
Sorry, @stevepiercy, I didn't look correctly; it's OK :-) |
|
@trollfot the Python 3.9 ends in one month, I think that we can completely remove it. |
|
MacOS still fails |
|
Some of macos pass :-) => add in |
|
It still has an issue in MoneyInputWidgetTests! |
Remove python badges : wrong versions
Updated python versions
|
Does anyone have objections to merging this? I updated http://5.75.157.57/ so everybody can have a look |
|
I've been ready for 2 years 👍 |
|
Do we create a version |
|
Yes. But for the final release we should cleanup the issues, docs, README a bit there are links that are outdated and so. |
|
@trollfot can you fix that branch so we can merge this too Pylons/deformdemo#129 |

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.