Skip to content
This repository has been archived by the owner on Aug 1, 2020. It is now read-only.

Add support for Django 3.0 #32

Merged
merged 3 commits into from
Mar 22, 2020
Merged

Add support for Django 3.0 #32

merged 3 commits into from
Mar 22, 2020

Conversation

MarkKoz
Copy link
Member

@MarkKoz MarkKoz commented Mar 21, 2020

django-crispy-forms has to be at least version 1.8 for Django 3.0 support, but I pinned it to 1.9 because it seems there were some import issues and deprecation warnings fixed in 1.9.

I haven't extensively tested this and don't really know how, so this has to wait for #31.

MarkKoz added 3 commits March 21, 2020 10:16
django-crispy-forms 1.9 only supports Django 2.2 and higher so as a
consequence this package had to drop support for versions below 2.2.
@MarkKoz MarkKoz added type: feature New feature or request priority: 2 - normal area: dependencies Related to package dependencies and management status: stalled Something is blocking further progress labels Mar 21, 2020
Copy link
Member

@lemonsaurus lemonsaurus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I say we just merge this and see what happens.

@lemonsaurus lemonsaurus marked this pull request as ready for review March 22, 2020 21:35
@lemonsaurus lemonsaurus merged commit c741d39 into master Mar 22, 2020
@keans
Copy link

keans commented Apr 22, 2020

I just tried to use the module with Django 3.0 and I noticed that the email input is not working as expected. It gets rendered to

<input type="email" name="email" maxlength="254" class="emailinput" id="id_email">

Though on the bulma webpage the CSS-class is solely "input". Could someone please check this?

Besides, django-crispy-bulma 0.3 seems not to be on pipy is the intentionally?

@MarkKoz
Copy link
Member Author

MarkKoz commented Apr 23, 2020

@keans we can keep track of that problem in #31. Regarding version 0.3, it's not on PyPI because our release pipeline is currently broken. We are working on fixing it.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area: dependencies Related to package dependencies and management priority: 2 - normal status: stalled Something is blocking further progress type: feature New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants