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

Refactor explore view #39

Merged
merged 2 commits into from
Mar 17, 2021
Merged

Refactor explore view #39

merged 2 commits into from
Mar 17, 2021

Conversation

ka7eh
Copy link
Contributor

@ka7eh ka7eh commented Mar 5, 2021

Description

The explore view is now fully refactored and uses the newer versions of React and OpenLayers. The PR fixes multiple functionality bugs and interaction issues too.

The bulk of the changes are in gd-geostreaming/src/containers/Explore. It also required some changes in the Map component in gd-core to enable some features like layer control for the explore view. gd-gltg is updated to use the new view, but not gd-glm and it needs to be configured for the new view. The rest of the projects are still using the old codebase and need their own tickets for an upgrade.

The best way to test this is by running gltg locally (npm run start -- gd-gltg), going to the explore view (/explore/all), and comparing it with the current view (https://greatlakestogulf.org/explore/all).

Check out the issue link to see a list of some of the map interaction stuff that are updated in the new view.

You might want to check out other views too and make sure none of them are broken because of the new changes.

Issue link:

#38

What is the current behavior?

What is the new behavior (if this is a feature change)?

Screenshots (if applicable)

Review Time Estimate

  • Immediately
  • Within one week
  • When possible

Types of changes (select all that applies)

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Improvement
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • My change requires a change to the documentation.
  • I have updated the CHANGELOG.md.
  • This changeset requires updating dependencies.
  • I have updated the documentation accordingly.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

@ka7eh ka7eh requested a review from a team March 5, 2021 15:39
@aarajh
Copy link
Contributor

aarajh commented Mar 8, 2021

Looks good! I just noticed one small thing. When I select a sensor with more than 5 sensors (GREON-01), the sensor popup box is partially out of the view.
Screen Shot 2021-03-08 at 10 47 07 AM

@ka7eh
Copy link
Contributor Author

ka7eh commented Mar 8, 2021

Looks good! I just noticed one small thing. When I select a sensor with more than 5 sensors (GREON-01), the sensor popup box is partially out of the view.

This should be fixed in b66475c.

@max-zilla max-zilla merged commit 8bc659e into develop Mar 17, 2021
@ka7eh ka7eh deleted the refactor-explore-view branch March 17, 2021 19:19
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.

3 participants