Skip to content
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

Add support for <optgroup> in SelectField #667

Merged
merged 8 commits into from
Sep 9, 2021
Merged

Conversation

Giandrop
Copy link
Contributor

Add support for in SelectField as described in issue #656

@davidism
Copy link
Member

Please rebase this onto master, 2.3.x will not get another feature release.

wtforms/fields/core.py Outdated Show resolved Hide resolved
wtforms/fields/core.py Outdated Show resolved Hide resolved
wtforms/fields/core.py Outdated Show resolved Hide resolved
@Giandrop Giandrop changed the base branch from 2.3.x to master October 12, 2020 03:09
@azmeuk azmeuk added the enhancement New feature, or existing feature improvement label Oct 12, 2020
@azmeuk
Copy link
Member

azmeuk commented Oct 13, 2020

Thank you for your contribution. You still have some tests failing, and some style issues.

@azmeuk azmeuk linked an issue Nov 4, 2020 that may be closed by this pull request
@azmeuk
Copy link
Member

azmeuk commented May 19, 2021

@Giandrop are you still interested in solving the issues related to this PR?

@Giandrop
Copy link
Contributor Author

Sorry, I completly forgot about this. I will be revisiting it during this weekend.

@Giandrop Giandrop marked this pull request as draft May 25, 2021 20:01
@Giandrop Giandrop marked this pull request as ready for review May 25, 2021 23:17
@Giandrop Giandrop requested a review from davidism May 28, 2021 03:22
@azmeuk azmeuk merged commit 77483ae into wtforms:master Sep 9, 2021
@azmeuk
Copy link
Member

azmeuk commented Sep 9, 2021

Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature, or existing feature improvement
Development

Successfully merging this pull request may close these issues.

SelectField <optgroup> support
3 participants