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

Add swingerclub #1187

Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

Add swingerclub #1187

wants to merge 6 commits into from

Conversation

Asteliks
Copy link

@Asteliks Asteliks commented Apr 2, 2024

Solves issue #1186

Copy link
Member

@tyrasd tyrasd left a comment

Choose a reason for hiding this comment

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

Hey. Usage is relatively low, but I guess it would be reasonable to include this as a preset, so that these amenities are not mistakenly mapped as strip clubs or the like.

Only one suggestion: I'm not sure if the fas-cocktail icon is a good fit here. Maybe a better icon would be something like maki-heart (used by the Love Hotel preset already) or fas-venus-mars?

data/presets/amenity/swingerclub.json Outdated Show resolved Hide resolved
Copy link

github-actions bot commented Apr 5, 2024

🍱 You can preview the tagging presets of this pull request here.

@Asteliks
Copy link
Author

Asteliks commented Apr 6, 2024

Hi, thanks for the fix. Also good point maki-heart seems best. I will change it as soon as I can

@Asteliks
Copy link
Author

@tyrasd is there anything else I need to do for us to merge this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants