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

Searching for space #1137

Merged
merged 9 commits into from
Apr 26, 2024
Merged

Searching for space #1137

merged 9 commits into from
Apr 26, 2024

Conversation

Adamik10
Copy link
Contributor

@Adamik10 Adamik10 commented Apr 19, 2024

Link to issue

https://reload.atlassian.net/browse/DDFLSBP-529

Description

This PR introduces handling of search results with the query not defined, or under 3 characters in length.

Screenshot of the result

Additional comments or questions

Sibling PR in the CMS

Copy link
Contributor

@spaceo spaceo left a comment

Choose a reason for hiding this comment

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

I have some change requests ;)

src/apps/search-result/helper.ts Outdated Show resolved Hide resolved
@Adamik10 Adamik10 requested a review from spaceo April 23, 2024 19:10
..component. Because of other refactorings, the content of the helper
function was miniscule, and as such can nicely fit into an if statement.
@spaceo spaceo removed their assignment Apr 24, 2024
Copy link
Contributor

@spaceo spaceo left a comment

Choose a reason for hiding this comment

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

We could put the min length conf in our core configuration.
But these changes makes me overwhelmingly happy.

@Adamik10 Adamik10 changed the base branch from release/brahma-15 to release/brahma-15-1 April 26, 2024 10:42
@Adamik10 Adamik10 merged commit d89d01d into release/brahma-15-1 Apr 26, 2024
20 checks passed
@Adamik10 Adamik10 deleted the searching-for-space branch April 26, 2024 10:43
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.

4 participants