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

[FLAG-844] Add tree cover density to analysis #4683

Merged

Conversation

willian-viana
Copy link
Collaborator

@willian-viana willian-viana commented Sep 4, 2023

Overview

It makes no sense to use the “tree cover by type” widget for both tree cover 2000/2010 data and tropical tree cover analyses. Taking that into consideration, researchers are ok with us instead using the widget you created, “Tree Cover Density in [area]“, for Tropical Tree Cover analysis.

From user stories, this should appear only at the country, region, and sub-region level, as well as for saved custom areas.

Demo

Screenshot 2023-09-07 at 17 49 47

Testing

  • Open the map.
  • Select an Admin or Custom Area.
  • Click to analyse the area
  • Select Tropical Tree cover Layer
  • Verify if the right widget (Tree Cover Density) is showing correctly.

@willian-viana willian-viana self-assigned this Sep 4, 2023
@willian-viana willian-viana changed the title feat(ttc): add tree cover density to analysis [FLAG-844] Add tree cover density to analysis Sep 4, 2023
@willian-viana willian-viana marked this pull request as ready for review September 4, 2023 15:03
@willian-viana willian-viana temporarily deployed to gfw-staging-pr-4683 September 4, 2023 15:09 Inactive
@willian-viana willian-viana temporarily deployed to gfw-staging-pr-4683 September 7, 2023 20:49 Inactive
@willian-viana willian-viana merged commit b985a8c into develop Sep 12, 2023
4 checks passed
@willian-viana willian-viana deleted the feat/add-tree-cover-density-to-ttc-layer-FLAG-844 branch September 12, 2023 14:38
@willian-viana willian-viana mentioned this pull request Sep 12, 2023
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