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

Feature/biodiversity glads aoi widget #3355

Merged
merged 15 commits into from
Apr 4, 2018

Conversation

edbrett
Copy link
Contributor

@edbrett edbrett commented Mar 27, 2018

Overview

A new shiny widget to display the GLAD alerts within a country of region, broken down by sub regions, and ranked by total area or relative percentage of tree cover extent.

screen shot 2018-03-27 at 18 31 06

@edbrett edbrett requested review from pjosh and benlaken March 27, 2018 16:31
Copy link
Contributor

@pjosh pjosh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome! 👏 👏
Only a single comment about the weeks select data.
And I need the GLAD_PRECALC_DATASET value to test in my local 😉

@@ -1,4 +1,8 @@
[
{
"label": "4 weeks",
"value": 4
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this affects the glad-alerts widget. Is this right?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah cheese 🤔 . Yes I should have put in the .env. And yes weeks selector needs updating. 🏎 💨

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed!

@01painadam
Copy link
Contributor

In an attempt to speed things up a little I've added a further condition in the sql which only retrieves the last years worth of glad alerts - @edbrett / @pjosh , can you double check my syntax here please :)

I also added the widget to the conservation tab.

@edbrett
Copy link
Contributor Author

edbrett commented Mar 29, 2018

We can review Monday morning as it doesnt have to be production until the afternoon. Provisionally looks good.

@01painadam
Copy link
Contributor

01painadam commented Mar 30, 2018

A few further updates based on feedback:

  • changes alerts_count to alerts in sql following table update
  • Added total count in place of area% to dynamic sentence
  • Fixes small bug in dynamic sentence where area value would change with unit

Other unimplemented feedback for future reference:

  • having different default indicator depending on tab (unsure how to do this nicely)
  • including meta data (key_biodiversity_areas)

@edbrett edbrett force-pushed the feature/biodiversity-glads-aoi-widget branch from 9a41e09 to 148f246 Compare April 2, 2018 07:55
@01painadam
Copy link
Contributor

  • Removes Glad-Biodiversity from Summary tab as requested
  • Adds missing locations to GAIN widgets

@edbrett edbrett merged commit 4035253 into develop Apr 4, 2018
@edbrett edbrett deleted the feature/biodiversity-glads-aoi-widget branch April 4, 2018 15:41
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.

None yet

3 participants