Skip to content
This repository has been archived by the owner on Mar 4, 2020. It is now read-only.

Feature/new carbon layers #431

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from
Open

Feature/new carbon layers #431

wants to merge 2 commits into from

Conversation

01painadam
Copy link
Contributor

Adds Carbon Emissions layer and legend to Carbon Loss tab.

@01painadam 01painadam requested a review from edbrett April 12, 2018 12:49
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.2%) to 30.029% when pulling 148c087 on feature/new-carbon-layers into ba22b76 on develop.

edbrett
edbrett approved these changes Apr 12, 2018
View reviewed changes
Copy link
Contributor

@edbrett edbrett left a comment

Choose a reason for hiding this comment

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

Looks good!

@simaob
Copy link
Contributor

simaob commented Apr 12, 2018

I think that maybe we don't merge this just yet. But deploy this branch to staging for them to review.

@simaob simaob temporarily deployed to gfwc-staging April 12, 2018 14:59 Inactive
@benlaken
Copy link
Contributor

Just pinged the clients about these layers on staging again, seems they still haven't reviewed it

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@edbrett edbrett edbrett approved these changes

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
STAGING WIP
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@01painadam @coveralls @simaob @benlaken @edbrett @j8seangel