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

Leaving a field blank omits it from the config #15

Closed
simonw opened this issue Nov 17, 2023 · 3 comments
Closed

Leaving a field blank omits it from the config #15

simonw opened this issue Nov 17, 2023 · 3 comments
Labels
bug Something isn't working

Comments

@simonw
Copy link
Collaborator

simonw commented Nov 17, 2023

CleanShot 2023-11-16 at 20 55 13@2x

Resulted in this in the database

CleanShot 2023-11-16 at 20 54 40@2x

Also, need to figure out that thing where we store as {"key": ["list of one value"]} - can we be neater than that while still supporting multiple items?

@simonw simonw added the bug Something isn't working label Nov 17, 2023
@simonw
Copy link
Collaborator Author

simonw commented Nov 24, 2023

Also, need to figure out that thing where we store as {"key": ["list of one value"]} - can we be neater than that while still supporting multiple items?

I fixed that in:

@simonw
Copy link
Collaborator Author

simonw commented Nov 24, 2023

Weirdly at the moment it looks like setting the field to blank uses the default value for the field instead:

@simonw
Copy link
Collaborator Author

simonw commented Nov 28, 2023

This is working OK now.

@simonw simonw closed this as completed Nov 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant