Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[feat] Status Filter Functionality #94

Merged
merged 4 commits into from
Dec 6, 2024

Conversation

nehaahussain
Copy link
Collaborator

@nehaahussain nehaahussain commented Dec 4, 2024

What's new in this PR

Description

  • this pr has changes from ethan's technology filter functionality sprint
  • filter by status: proposed or operational
  • fix bug when there are no projects to display from filters

Screenshots

Screen.Recording.2024-12-03.at.10.21.13.PM.mov

How to review

  • map view screen

Next steps

  • clearFilters function in map view screen currently clears all filters -> do we wanna change it to clear different filters instead of all?

Relevant links

Online sources

Related PRs

CC: @itsliterallymonique

@nehaahussain nehaahussain linked an issue Dec 4, 2024 that may be closed by this pull request
@nehaahussain nehaahussain marked this pull request as ready for review December 4, 2024 06:29
Copy link
Collaborator

@itsliterallymonique itsliterallymonique left a comment

Choose a reason for hiding this comment

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

Great job. You can squash and merge

@nehaahussain nehaahussain merged commit b4fe69c into main Dec 6, 2024
2 checks passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[feat] Status Filter Functionality
2 participants