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-721] Fix the integrated deforestation alerts widget in AOI view #4537

Merged
merged 2 commits into from
Apr 19, 2023

Conversation

willian-viana
Copy link
Collaborator

@willian-viana willian-viana commented Apr 7, 2023

Overview

The integrated deforestation alerts widget isn’t showing for this AOI on the map or dashboard: Subri River Deforestation Rates & Statistics | GFW. Here is the link to the widget: Subri River Deforestation Rates & Statistics | GFW.

Demo

image-20230320-215653

Notes

According to https://resource-watch.github.io/doc-api/reference.html\#what-are-geostore-objects hash and id are the same type and value

Testing

  • Open the AOI link and try to see the widget data

@willian-viana willian-viana self-assigned this Apr 7, 2023
@pedrogpimenta pedrogpimenta temporarily deployed to gfw-staging-pr-4537 April 7, 2023 20:23 Inactive
@willian-viana willian-viana changed the title fix(integrated-deforestation): using geostore id over hash [FLAG-722] Fixthe integrated deforestation alerts widget in AOI view Apr 7, 2023
@willian-viana willian-viana changed the title [FLAG-722] Fixthe integrated deforestation alerts widget in AOI view [FLAG-722] Fix the integrated deforestation alerts widget in AOI view Apr 7, 2023
@pedrogpimenta pedrogpimenta temporarily deployed to gfw-staging-pr-4537 April 7, 2023 20:39 Inactive
@willian-viana willian-viana marked this pull request as ready for review April 7, 2023 20:41
@willian-viana willian-viana changed the title [FLAG-722] Fix the integrated deforestation alerts widget in AOI view [FLAG-721] Fix the integrated deforestation alerts widget in AOI view Apr 7, 2023
@willian-viana willian-viana force-pushed the fix/integrated-deforastion-alerts-FLAG-721 branch from 6a522cf to fcca5c3 Compare April 7, 2023 20:53
@pedrogpimenta pedrogpimenta temporarily deployed to gfw-staging-pr-4537 April 7, 2023 20:54 Inactive
@wri7tno
Copy link
Collaborator

wri7tno commented Apr 11, 2023

LGTM, looks cleaner 👍

@willian-viana willian-viana force-pushed the fix/integrated-deforastion-alerts-FLAG-721 branch from fcca5c3 to ad152be Compare April 17, 2023 19:21
@pedrogpimenta pedrogpimenta temporarily deployed to gfw-staging-pr-4537 April 17, 2023 19:22 Inactive
@willian-viana willian-viana removed the request for review from SARodrigues April 18, 2023 15:36
@willian-viana willian-viana temporarily deployed to gfw-staging-pr-4537 April 19, 2023 15:59 Inactive
@willian-viana willian-viana merged commit 21dddd3 into develop Apr 19, 2023
@willian-viana willian-viana deleted the fix/integrated-deforastion-alerts-FLAG-721 branch April 19, 2023 16:20
@willian-viana willian-viana mentioned this pull request Apr 19, 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.

3 participants