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

Get Filters Working #475

Merged
merged 7 commits into from
Jul 31, 2024
Merged

Get Filters Working #475

merged 7 commits into from
Jul 31, 2024

Conversation

tayloraj10
Copy link
Contributor

Pull Request

Change Summary

Summary of changes made. This can be a paragraph, list of bullet points, etc. Aim to be brief and concise.

Change Reason

Summary of why changes are being made. Link to other documents, Slack conversations, etc are good to post here for explaining the reason for particular changes.

Verification [Optional]

A .gif, video, or screenshots of the feature are extremely helpful in verifying that code changes are working correctly. Feel free to include them in the PR to help the reviewer.

We recommend using OBS to record videos, as it is open source, free, and available on all major operating systems.

Related Issue: #<Github Issue Number (if one exists)>

@tayloraj10 tayloraj10 requested a review from vontell July 14, 2024 19:13
@vontell
Copy link
Contributor

vontell commented Jul 24, 2024

Hey @tayloraj10! Is this ready to review? Just wanted to confirm the state of this PR before jumping in

@tayloraj10
Copy link
Contributor Author

Hey @tayloraj10! Is this ready to review? Just wanted to confirm the state of this PR before jumping in

@vontell yeah this is ready for review

@tayloraj10 tayloraj10 changed the title [Dont Merge] Get Filters Working Get Filters Working Jul 30, 2024
cypress/e2e/desktop/filters.cy.js Outdated Show resolved Hide resolved
src/actions/actions.js Outdated Show resolved Hide resolved
src/components/Filter/Filter.js Outdated Show resolved Hide resolved
src/components/ReactGoogleMaps/ReactGoogleMaps.js Outdated Show resolved Hide resolved
src/reducers/filterMarkers.js Outdated Show resolved Hide resolved
src/reducers/filterMarkers.js Outdated Show resolved Hide resolved
src/selectors/resourceSelectors.js Show resolved Hide resolved
src/selectors/resourceSelectors.js Outdated Show resolved Hide resolved
@tayloraj10 tayloraj10 requested a review from vontell July 31, 2024 00:03
@tayloraj10 tayloraj10 merged commit 36a8e13 into develop Jul 31, 2024
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants