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

displaying choropleth from the rich data panel #315

Closed
wants to merge 27 commits into from
Closed

displaying choropleth from the rich data panel #315

wants to merge 27 commits into from

Conversation

emre2038
Copy link
Contributor

@emre2038 emre2038 commented Apr 15, 2021

Description

Displaying choropleth of a subindicator on a chart on the rich data panel

Related Issue

#281

How to test it locally

Screenshots

Changelog

Added

Updated

Removed

Checklist

  • 🚀 is the code ready to be merged and go live?
  • 🛠 does it work (build) locally
  • 👩‍🎨 does the design matches the Demo

Pull Request

  • 📰 good title
  • 📝good description
  • 🔖 issue linked
  • 📖 changelog filled out
  • commit messages are meaningful

Code Quality

  • 🚧 no commented out code
  • 🖨 no unnecessary logging
  • 🎱 no magic numbers

Testing

  • ✅ added (appropriate) unit tests
  • 💢 edge cases in tests were considered
  • ✅ ran tests locally & are passing

emre2038 and others added 27 commits March 23, 2021 13:46
use either the local server or the actual one (once the changes are in
place) to test this out. removed the hard-coded values.
also removed and changed code that will need to be updated as well, as
we're now using the new data format for the population of the data
mapper as well
there is some work left for making sure when a geography changes that
the data is properly used
removed the on-the-fly getting of the indicator data for now until the
BE is ready. the data is available in the all_details request anyway for
now but this will change in the future
… on_the_fly_choropleth_data

# Conflicts:
#	server.js
slightly adjust the subindicator to not pass too many variables along
that only change once and can be saved to the class

let result = bt.calculate({ data, subindicatorArr })
const CHILD_DATA = {
EC: [{
Copy link

Choose a reason for hiding this comment

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

Similar blocks of code found in 2 locations. Consider refactoring.

gender: "Female",
language: "English"
}],
FS: [{
Copy link

Choose a reason for hiding this comment

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

Similar blocks of code found in 2 locations. Consider refactoring.

@emre2038 emre2038 self-assigned this Apr 15, 2021
@emre2038 emre2038 added this to the 0.14 milestone Apr 15, 2021
@emre2038 emre2038 linked an issue Apr 15, 2021 that may be closed by this pull request
@netlify
Copy link

netlify bot commented Apr 15, 2021

Deploy preview for wazimap-staging ready!

Built with commit c4a3c16

https://deploy-preview-315--wazimap-staging.netlify.app

@emre2038 emre2038 closed this May 5, 2021
@emre2038 emre2038 deleted the display_choropleth_from_rdv branch May 5, 2021 20:31
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