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

[search] Do not emit private swimming pools. #7870

Merged
merged 2 commits into from
Apr 9, 2024
Merged

[search] Do not emit private swimming pools. #7870

merged 2 commits into from
Apr 9, 2024

Conversation

vng
Copy link
Member

@vng vng commented Apr 8, 2024

Closes #7867
Closes #3269

vng added 2 commits April 8, 2024 15:16
Signed-off-by: Viktor Govako <viktor.govako@gmail.com>
Signed-off-by: Viktor Govako <viktor.govako@gmail.com>
@vng vng added the Search Search label Apr 8, 2024
@vng vng requested a review from biodranik April 8, 2024 18:49
@@ -8049,6 +8049,10 @@ sw:Bwawa la kuogelea
fa:استخرشنا
mr:जलतरण तलाव

# Not searchable
leisure-swimming_pool-private
en:DoNotMatchThisType_OneLanguageDefinitionIsEnoughHere
Copy link
Member

Choose a reason for hiding this comment

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

What a hack! Why not searchable types should be added to categories.txt? It would be a hardly supported mess soon. There are types in categories that are searchable, and non-searchable, and there are non-searchable types not in categories...

Is it hard to keep the rule "if present in categories.txt then it's indexed and is searchable"?

Copy link
Member Author

@vng vng Apr 8, 2024

Choose a reason for hiding this comment

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

This is not a hack. We do search subtypes when the parent type is defined. To concrete subtype behaviour, need to define it manually.

Copy link
Member

Choose a reason for hiding this comment

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

It looks like a hack. Does it make sense to document that behavior in categories.txt?

Copy link
Member Author

Choose a reason for hiding this comment

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

Comment about child-parent matching is already present at the beginning of this file.

drape_frontend/selection_shape_generator.cpp Show resolved Hide resolved
@vng vng merged commit d2f16d5 into master Apr 9, 2024
15 checks passed
@vng vng deleted the vng-search branch April 9, 2024 12:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Search Search
Projects
None yet
2 participants