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

Case insensitive for model label #5

Open
sveetch opened this issue Mar 24, 2024 · 0 comments
Open

Case insensitive for model label #5

sveetch opened this issue Mar 24, 2024 · 0 comments
Labels
idea Idea about feature or new concept

Comments

@sveetch
Copy link
Member

sveetch commented Mar 24, 2024

Is your feature request related to a problem? Please describe.

Currently inclusion and exclusion definitions are case sensitive on model label: auth.user won't work, auth.User is valid.

Describe the solution you'd like

This may be simpler to have case insensitive. I don't think there is added value to keep case sensitive.

It has been developed with case sensitive because it was not clear if case mismatching could happen with between models from applications, however Django itself in dumpdata manage case insensitive so it seems we could stand on this.

This means we probably have to lowercase everything from appstore. This change should be retro compatible with definitions written with case sensitive.

@sveetch sveetch added the idea Idea about feature or new concept label Mar 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
idea Idea about feature or new concept
Projects
None yet
Development

No branches or pull requests

1 participant