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

feat: make redirects via the input component case-insensitive #875

Merged
merged 3 commits into from Feb 8, 2023

Conversation

jkaho
Copy link
Contributor

@jkaho jkaho commented Feb 7, 2023

Assume users don't care about the case-sensitivity of redirects and lowercase all inputs when retrieving redirects, as well as the redirect keys themselves (the queries that trigger the redirects).

DAU-249

@changeset-bot
Copy link

changeset-bot bot commented Feb 7, 2023

🦋 Changeset detected

Latest commit: cca168c

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 2 packages
Name Type
@sajari/react-search-ui Minor
sajari-sdk-docs Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@jkaho jkaho force-pushed the DAU-249/lowercase-input-to-look-up-redirects branch from 63fc196 to 2e1858c Compare February 7, 2023 03:40
@jkaho jkaho force-pushed the DAU-249/lowercase-input-to-look-up-redirects branch from 2e1858c to 9966eb0 Compare February 7, 2023 03:56
Copy link
Member

@jonathaningram jonathaningram left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

RCFV

@jkaho jkaho force-pushed the DAU-249/lowercase-input-to-look-up-redirects branch from 013e4ae to fe9ea75 Compare February 7, 2023 05:45
@jkaho jkaho force-pushed the DAU-249/lowercase-input-to-look-up-redirects branch 2 times, most recently from d639f4d to 918b429 Compare February 7, 2023 06:54
Copy link
Member

@jonathaningram jonathaningram left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Small doc nits. Good to go otherwise.

docs/pages/search-ui/input.mdx Outdated Show resolved Hide resolved
docs/pages/search-ui/input.mdx Outdated Show resolved Hide resolved
@jkaho jkaho force-pushed the DAU-249/lowercase-input-to-look-up-redirects branch from 918b429 to feb83e2 Compare February 8, 2023 00:33
jonathaningram
jonathaningram previously approved these changes Feb 8, 2023
@jkaho jkaho changed the title feat: lowercase search input value when looking for redirects feat: make redirects via the input component case-insensitive Feb 8, 2023
@jkaho jkaho force-pushed the DAU-249/lowercase-input-to-look-up-redirects branch 4 times, most recently from 310397a to 9ec2442 Compare February 8, 2023 01:39
@jkaho
Copy link
Contributor Author

jkaho commented Feb 8, 2023

@jonathaningram sorry needed to rejig the test since it was passing locally but failing in the CI check

@jkaho jkaho force-pushed the DAU-249/lowercase-input-to-look-up-redirects branch from 9ec2442 to cca168c Compare February 8, 2023 02:26
@jkaho jkaho merged commit 30eefe2 into master Feb 8, 2023
@jkaho jkaho deleted the DAU-249/lowercase-input-to-look-up-redirects branch February 8, 2023 02:39
jkaho added a commit that referenced this pull request Feb 8, 2023
jkaho added a commit that referenced this pull request Feb 8, 2023
#877)

This reverts commit 30eefe2 / #875.

Reason being I made it a minor bump when it should probably be major, considering that the changes to the component may change the current behaviour of users' redirects if they rely on case-sensitivity of their redirects.
jkaho added a commit that referenced this pull request Feb 8, 2023
Revert "Revert "feat: make redirects via the input component case-insensitive" (#877)"

This reverts commit f2279a4.

Take 2 of #875.

DAU-249
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.

None yet

2 participants