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 app state #1133

Merged
merged 14 commits into from
Mar 10, 2022
Merged

Add app state #1133

merged 14 commits into from
Mar 10, 2022

Conversation

hanapotski
Copy link
Contributor

@hanapotski hanapotski commented Dec 10, 2021

Fixes #989
Fixes #1101

@hanapotski hanapotski marked this pull request as ready for review January 30, 2022 01:03
Add site context and refactor app state

Fix server error

Remove showSearchIcon prop

Make recenter button work

Update verification admin

Move selectedOrganization to app state

Move userCoordinates to app state

Make deep linking work 🙌

Fix tiny bugs

Use siteContext and do some refactoring

Refactor some components
@hanapotski
Copy link
Contributor Author

@entrotech don't merge it yet, I just found a bug 😅

@entrotech entrotech merged commit 72ef778 into develop Mar 10, 2022
@entrotech
Copy link
Member

I needed to merge, so I could integrate your changes with Issue #1165. I won't deploy to production yet, so we can fix whatever bugs we find.

@hanapotski hanapotski mentioned this pull request Mar 12, 2022
@entrotech entrotech deleted the 989-deep-linking branch March 12, 2023 22:11
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.

Bug: GPS / Use My Location button is disabled on map Bug: Deep linking on the Map Page is broken
3 participants