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

Bug: Search form issue - Search should be case-insensitive #567

Closed
audserraCGI opened this issue Apr 8, 2022 · 3 comments
Closed

Bug: Search form issue - Search should be case-insensitive #567

audserraCGI opened this issue Apr 8, 2022 · 3 comments
Labels
bug Something isn't working help wanted Extra attention is needed

Comments

@audserraCGI
Copy link
Contributor

Expected Behavior

When using the search form to filter the device list, searching by name/ID should be case-insensitive.

Current Behavior

Searching "device" doesn't allow us to retrieve our mock device called "Device 1"

Additional Information

There's also a pragma warning in the code asking to normalize the filtering:

#pragma warning disable CA1308 // Normalize strings to uppercase
filter += $" AND (STARTSWITH(deviceId, '{searchText.ToLowerInvariant()}') OR (is_defined(tags.deviceName) AND STARTSWITH(tags.deviceName, '{searchText}')))";
#pragma warning restore CA1308 // Normalize strings to uppercase

@audserraCGI audserraCGI added the bug Something isn't working label Apr 8, 2022
@kbeaugrand kbeaugrand added this to the S19 milestone Apr 8, 2022
@kbeaugrand kbeaugrand moved this to :pencil: Todo in IoT Hub Portal Apr 8, 2022
@kbeaugrand kbeaugrand removed this from the S19 milestone Apr 12, 2022
@kbeaugrand
Copy link
Member

Currently, IoT Hub query language doesn't provide functions that can be used to query easily the devices (with lowercase, starts_with, contains, ...).

We decided to postpone this implementation until these features are available in IoT Hub query language.

Se https://docs.microsoft.com/en-US/azure/iot-hub/iot-hub-devguide-query-language

@kbeaugrand
Copy link
Member

Blocked by #1029, #1030, and #1031

@hocinehacherouf
Copy link
Contributor

Fix by #862

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working help wanted Extra attention is needed
Projects
Status: 📝 Todo
Development

No branches or pull requests

3 participants