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

[MRXN23-488]: map legend: merges binary and continuous features #1552

Merged
merged 2 commits into from
Oct 26, 2023

Conversation

andresgnlez
Copy link
Member

Groups binary and continuous features in map legend.

Overview

This PR groups binary and continuous features in the map legend. The criteria to difference between binary and continuous is checking the properties min and max in every feature. These properties are not implemented in the API yet so it's not possible to distinguish between both types of properties right now.

Designs

Testing instructions

Please explain how to test the PR: ID of a dataset, steps to reach the feature, etc.

Feature relevant tickets

https://vizzuality.atlassian.net/browse/MRXN23-488


Checklist before submitting

  • Meaningful commits and code rebased on develop.
  • If this PR adds feature that should be tested for regressions when
    deploying to staging/production, please add brief testing instructions
    to the deploy checklist (docs/deployment-checklist.md)
  • Update CHANGELOG file

@andresgnlez andresgnlez added the Frontend Everything related frontend label Oct 17, 2023
@andresgnlez andresgnlez self-assigned this Oct 17, 2023
@vercel
Copy link

vercel bot commented Oct 17, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
marxan ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 26, 2023 11:59am

@andresgnlez andresgnlez marked this pull request as ready for review October 26, 2023 10:14
@andresgnlez andresgnlez changed the base branch from MRXN23-458-fe-implementation-of to develop October 26, 2023 10:16
@andresgnlez andresgnlez force-pushed the MRXN23-488-fe-binary-raw-vs branch from 3c33209 to ebf51e3 Compare October 26, 2023 11:57
@andresgnlez andresgnlez merged commit 00de4d4 into develop Oct 26, 2023
61 checks passed
@andresgnlez andresgnlez deleted the MRXN23-488-fe-binary-raw-vs branch October 26, 2023 12:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Frontend Everything related frontend
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant