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

Confusing case requirements #2581

Open
chrisys opened this issue Jan 13, 2023 · 4 comments
Open

Confusing case requirements #2581

chrisys opened this issue Jan 13, 2023 · 4 comments

Comments

@chrisys
Copy link
Member

chrisys commented Jan 13, 2023

Screenshot 2023-01-13 at 09 46 54

It seems the CLI is case sensitive when pushing - i.e. 'balena push BarnFin' works whilst 'balena push barnfin' does not, which isn't necessarily an issue, but then the first like of the info says 'building for barnfin' which is in direct conflict with the message above.

I'd suggest first we stick to either case sensitive or not, and then also provide a hint to the user if the application isn't found. We just say 'nope' when we could say 'did you mean BarnFin?' or even 'note: slugs/fleet names are case sensitive'

@myarmolinsky
Copy link
Member

myarmolinsky commented Aug 4, 2023

The SDK treats slugs as case-insensitive so the command should not be case-sensitive

@myarmolinsky
Copy link
Member

The SDK treats slugs as case-insensitive so the command should not be case-sensitive

Actually while this is true, you passed an app name, not a slug. Slugs are treated as case-insensitive but app names are sent through a different method which apparently does care about the case-sensitivity. Need to figure out why we care about the case for app name

@myarmolinsky
Copy link
Member

myarmolinsky commented Aug 4, 2023

Apparently we've got some confusing UX in the dashboard related to this...
Here's me trying to create a fleet with the name asdas in an org that already has a fleet with the name asdas:
image
As you can see, validation tells the user that the name is case-insensitive. So if I enter asdaS instead, the error goes away. However, trying to submit results in the following:
image

So according to the API, app names are case-insensitive: if you have asdas, you cannot have asdaS. So the dashboard should validate with disregard for case.

We have some confusing UX from the API too though:
image

So although application names are treated case-insensitively, we still disallow getting application names with case-insensitivity ...

@myarmolinsky
Copy link
Member

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants