Skip to content

Show empty default#2501

Closed
m-freiberg wants to merge 8 commits into
pallets:stablefrom
m-freiberg:show_empty_default
Closed

Show empty default#2501
m-freiberg wants to merge 8 commits into
pallets:stablefrom
m-freiberg:show_empty_default

Conversation

@m-freiberg

Copy link
Copy Markdown

Use None instead of "" to indicate that default_string is unset to allow for empty string defaults.

#2500

New behavior:
image

Checklist:

  • Add tests that demonstrate the correct behavior of the change. Tests should fail without the change.
  • Add or update relevant docs, in the docs folder and in code.
  • Add an entry in CHANGES.rst summarizing the change and linking to the issue.
  • Add .. versionchanged:: entries in any relevant code docs.
  • Run pre-commit hooks and fix any issues.
  • Run pytest and tox, no tests failed.

@davidism davidism added this to the 8.2.0 milestone Jul 4, 2023
@AndreasBackx AndreasBackx mentioned this pull request Oct 20, 2024
34 tasks
@AndreasBackx

Copy link
Copy Markdown
Collaborator

Fixed by #2724.

@AndreasBackx AndreasBackx removed this from the 8.2.0 milestone Oct 26, 2024
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Nov 10, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants