Skip to content
This repository was archived by the owner on Feb 11, 2026. It is now read-only.

Comments

fix: choropleth statistics query with map bounds#224

Merged
joaquimds merged 1 commit intomainfrom
feature/map-1024-output-areas-not-displaying-choropleth-data
Feb 27, 2025
Merged

fix: choropleth statistics query with map bounds#224
joaquimds merged 1 commit intomainfrom
feature/map-1024-output-areas-not-displaying-choropleth-data

Conversation

@joaquimds
Copy link
Member

mapBounds needs to be in the top level of the query variables for fetchMore({ variables: { mapBounds } }) to work.

@linear
Copy link

linear bot commented Feb 26, 2025

@commonknowledge-bot commonknowledge-bot deployed to feature/map-1024-output-areas-not-displaying-choropleth-data - meep-database PR #224 February 26, 2025 16:23 — with Render Active
@sentry
Copy link

sentry bot commented Feb 26, 2025

🔍 Existing Issues For Review

Your pull request is modifying functions with the following pre-existing issues:

📄 File: hub/graphql/types/model_types.py

Function Unhandled Issue
statistics_for_choropleth GraphQLError: column hub_genericdata.data_EU Citizens - Total electors (includes attaine_5035d8b9 does not exist ...
Event Count: 100
statistics_for_choropleth GraphQLError: 'Percentage registered' /graphql
Event Count: 16
statistics_for_choropleth GraphQLError: 'variable' /graphql
Event Count: 5
📄 File: hub/graphql/types/stats.py (Click to Expand)
Function Unhandled Issue
statistics GraphQLError: column hub_genericdata.area_id does not exist ...
Event Count: 540
statistics GraphQLError: column hub_genericdata.data_EU Citizens - Total electors (includes attaine_5035d8b9 does not exist ...
Event Count: 100
statistics GraphQLError: 'Percentage registered' /graphql
Event Count: 16
statistics GraphQLError: 'variable' /graphql
Event Count: 12
---

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

mapBounds needs to be in the top level of the query variables for
fetchMore({ variables: { mapBounds } }) to work.
@joaquimds joaquimds force-pushed the feature/map-1024-output-areas-not-displaying-choropleth-data branch from 31fa119 to c455f27 Compare February 26, 2025 16:23
@commonknowledge-bot commonknowledge-bot temporarily deployed to feature/map-1024-output-areas-not-displaying-choropleth-data - pgbouncer PR #224 February 26, 2025 16:24 — with Render Destroyed
@janbaykara
Copy link
Member

Fine by me!

@commonknowledge-bot commonknowledge-bot temporarily deployed to feature/map-1024-output-areas-not-displaying-choropleth-data - meep-intelligence-hub-backend PR #224 February 26, 2025 16:25 — with Render Destroyed
@commonknowledge-bot commonknowledge-bot temporarily deployed to feature/map-1024-output-areas-not-displaying-choropleth-data - meep-intelligence-hub-backend PR #224 February 26, 2025 16:25 — with Render Destroyed
@commonknowledge-bot commonknowledge-bot temporarily deployed to feature/map-1024-output-areas-not-displaying-choropleth-data - meep-intelligence-hub-frontend PR #224 February 26, 2025 16:25 — with Render Destroyed
@commonknowledge-bot commonknowledge-bot temporarily deployed to feature/map-1024-output-areas-not-displaying-choropleth-data - meep-intelligence-hub-worker PR #224 February 26, 2025 16:25 — with Render Destroyed
@commonknowledge-bot commonknowledge-bot temporarily deployed to feature/map-1024-output-areas-not-displaying-choropleth-data - meep-intelligence-hub-frontend PR #224 February 26, 2025 16:25 — with Render Destroyed
@commonknowledge-bot commonknowledge-bot temporarily deployed to feature/map-1024-output-areas-not-displaying-choropleth-data - meep-intelligence-hub-backend PR #224 February 26, 2025 16:25 — with Render Destroyed
@joaquimds joaquimds merged commit 3e46372 into main Feb 27, 2025
5 of 6 checks passed
@joaquimds joaquimds deleted the feature/map-1024-output-areas-not-displaying-choropleth-data branch February 27, 2025 10:12
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants