Skip to content

QA remove lower case match#264

Merged
devbugging merged 1 commit into
release/v21from
qa/remove-case-insensitive
May 21, 2021
Merged

QA remove lower case match#264
devbugging merged 1 commit into
release/v21from
qa/remove-case-insensitive

Conversation

@devbugging
Copy link
Copy Markdown
Contributor

Remove to lower case when accessing map keys as some have multi-word keys that then don't work. Example parentId key can not be accesses as it is converted to parentid

@janezpodhostnik
Copy link
Copy Markdown
Contributor

You could also try both.
if value := jsonResult[filter] returns nothing
try value := jsonResult[strings.ToLower(filter)] as well

@devbugging devbugging merged commit a632107 into release/v21 May 21, 2021
@devbugging devbugging deleted the qa/remove-case-insensitive branch May 21, 2021 18:47
@devbugging
Copy link
Copy Markdown
Contributor Author

You could also try both.
if value := jsonResult[filter] returns nothing
try value := jsonResult[strings.ToLower(filter)] as well

ohhh right, smart

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

Successfully merging this pull request may close these issues.

2 participants