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: add tags to map filters #3991

Merged
merged 1 commit into from
Nov 11, 2024

Conversation

benfurber
Copy link
Member

PR Checklist

PR Type

What kind of change does this PR introduce?

  • Bugfix (fixes an issue)
  • Feature (adds functionality)
  • Code style update
  • Refactoring (no functional changes)
  • Build related changes
  • CI related changes
  • Documentation changes
  • Other... Please describe:

What is the current behavior?

What is the new behavior?

So far...

Screen.Recording.2024-11-07.at.16.01.24.mov

@benfurber benfurber force-pushed the feat/map/finish-filtering branch 4 times, most recently from b539f0f to 6028cb2 Compare November 7, 2024 16:25
@davehakkens
Copy link
Contributor

davehakkens commented Nov 7, 2024

will this show 'all activities tags' because at this point we would mainly need the ones from the spaces, the ones from individual members could be added later..

@benfurber
Copy link
Member Author

@davehakkens It filters out unused tags. I.e. it only shows the filters for the pins that exist. So until a tag is used, it won't be on the list.

Copy link

cypress bot commented Nov 11, 2024

onearmy-community-platform    Run #6541

Run Properties:  status check passed Passed #6541  •  git commit 6f577aa823: feat: add tags to map filters
Project onearmy-community-platform
Branch Review pull/3991
Run status status check passed Passed #6541
Run duration 05m 30s
Commit git commit 6f577aa823: feat: add tags to map filters
Committer Ben Furber
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 2
Tests that did not run due to a developer annotating a test with .skip  Pending 0
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 74
View all changes introduced in this branch ↗︎

@benfurber benfurber force-pushed the feat/map/finish-filtering branch 2 times, most recently from 8c22fa4 to a3f85ee Compare November 11, 2024 14:41
@benfurber benfurber marked this pull request as ready for review November 11, 2024 14:41
@benfurber benfurber requested a review from a team as a code owner November 11, 2024 14:41
@benfurber benfurber merged commit c553efd into ONEARMY:master Nov 11, 2024
18 of 19 checks passed
@benfurber benfurber deleted the feat/map/finish-filtering branch November 11, 2024 16:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants