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

[feature request] Hide "member" mappins by default on PP #2927

Closed
davehakkens opened this issue Oct 19, 2023 · 5 comments · Fixed by #2975 or #3000
Closed

[feature request] Hide "member" mappins by default on PP #2927

davehakkens opened this issue Oct 19, 2023 · 5 comments · Fixed by #2975 or #3000
Assignees
Labels

Comments

@davehakkens
Copy link
Contributor

Is your feature request related to a problem? Please describe.
The map is getting crowded (cluttered) which makes it harder for workspaces to pop out + slow loading times.

afbeelding

Describe the solution you'd like
We would like to hide the "want to get started" pins by default on Precious Plastic map

Notes
This is a specific setting for Precious Plastic with a mature community.
Other projects (instances) probably dont want to hide this in their early stages.
So ideally it's a setting that can be configured per instance.

@benfurber
Copy link
Member

@davehakkens It's fairly straightforward to add a function to how filtersSelected is loaded on the map Controls (src/pages/Maps/Content/Controls/Controls.tsx). And only for PP is straightforward. Making it an admin controlled configuration option is a lot more work.

As sorting it for PP only as a small task and as a admin (i.e. non-code) configuration is a big one, is there a priority difference for you on getting this work done?

@davehakkens
Copy link
Contributor Author

good to hear its fairly straightforward @benfurber .
For now we just need to have it configured for PP. No admin interface needed.

@onearmy-bot
Copy link
Collaborator

🎉 This issue has been resolved in version 1.120.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@benfurber
Copy link
Member

Doesn't look like #2975 worked. So the method used to switch instances must be different from how it's worked out during build. I'll try another approach.

@onearmy-bot
Copy link
Collaborator

🎉 This issue has been resolved in version 1.127.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Archived in project
3 participants