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/map 922 aggregation options for displays #196

Merged
merged 8 commits into from
Jan 24, 2025

Conversation

janbaykara
Copy link
Member

Based on work with the Migrant Democracy Project's recent data ingests, this PR does a few things to the Area Explorer display queries and UI:

  • Gives control over to the user to decide how to rollup/drilldown data. Just this area, stats in child areas, pull in parent area data? Etc. etc. This should fix MAP-926
  • An explicit display option for displaying aggregated data, or breakdown data of individual records with a carousel.
  • This PR also fixes the breadcrumb data issue; closes MAP-869

For now, query and display options are not saved. This is awaiting the refactor of config options in MAP-923.

@commonknowledge-bot commonknowledge-bot deployed to feature/map-922-aggregation-options-for-displays - meep-database PR #196 January 24, 2025 18:30 — with Render Active
@commonknowledge-bot commonknowledge-bot temporarily deployed to feature/map-922-aggregation-options-for-displays - meep-intelligence-hub-backend PR #196 January 24, 2025 18:31 — with Render Destroyed
@commonknowledge-bot commonknowledge-bot temporarily deployed to feature/map-922-aggregation-options-for-displays - meep-intelligence-hub-backend PR #196 January 24, 2025 18:31 — with Render Destroyed
@commonknowledge-bot commonknowledge-bot temporarily deployed to feature/map-922-aggregation-options-for-displays - meep-intelligence-hub-frontend PR #196 January 24, 2025 18:31 — with Render Destroyed
@commonknowledge-bot commonknowledge-bot had a problem deploying to feature/map-922-aggregation-options-for-displays - meep-intelligence-hub-worker PR #196 January 24, 2025 18:31 — with Render Failure
@commonknowledge-bot commonknowledge-bot temporarily deployed to feature/map-922-aggregation-options-for-displays - meep-intelligence-hub-frontend PR #196 January 24, 2025 18:31 — with Render Destroyed
@commonknowledge-bot commonknowledge-bot temporarily deployed to feature/map-922-aggregation-options-for-displays - meep-intelligence-hub-backend PR #196 January 24, 2025 18:31 — with Render Destroyed
@commonknowledge-bot commonknowledge-bot temporarily deployed to feature/map-922-aggregation-options-for-displays - meep-intelligence-hub-frontend PR #196 January 24, 2025 18:33 — with Render Destroyed
@commonknowledge-bot commonknowledge-bot temporarily deployed to feature/map-922-aggregation-options-for-displays - meep-intelligence-hub-backend PR #196 January 24, 2025 18:35 — with Render Destroyed
@commonknowledge-bot commonknowledge-bot temporarily deployed to feature/map-922-aggregation-options-for-displays - meep-intelligence-hub-frontend PR #196 January 24, 2025 18:35 — with Render Destroyed
@commonknowledge-bot commonknowledge-bot temporarily deployed to feature/map-922-aggregation-options-for-displays - meep-intelligence-hub-worker PR #196 January 24, 2025 18:35 — with Render Destroyed
@janbaykara janbaykara merged commit 5a243a1 into main Jan 24, 2025
3 of 4 checks passed
@janbaykara janbaykara deleted the feature/map-922-aggregation-options-for-displays branch January 24, 2025 18:46
Copy link

sentry-io bot commented Jan 31, 2025

Suspect Issues

This pull request was deployed and Sentry observed the following issues:

  • ‼️ GraphQLError: Failed to geocode postcode: DN8 4BU. /graphql View Issue
  • ‼️ GraphQLError: get() returned more than one Area -- it returned 2! /graphql View Issue

Did you find this useful? React with a 👍 or 👎

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