Drop the unused x and y columns from API v2's UK area impact tables
Les mainteneurs répondent en général sous 2 jours
Évaluation
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Accessibilité débutants
- 35/100
Piste de recherche
Start with the SQLModel tables in policyengine_api/data/v2/models/reports.py and the baseline revision f5ef4347cb2a; check how v2 Alembic revisions are generated and how their downgrade functions are written. Remove x and y from both tables and add an autogenerated revision that drops those columns and restores them on downgrade. Done means the models and revision match the intended schema; deployment is scheduled after the UK Budget on 28 October 2026 and requires Anthony's approval.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
API v2's constituency_impacts and local_authority_impacts tables (the ConstituencyImpact and LocalAuthorityImpact SQLModel tables in policyengine_api/data/v2/models/reports.py) each have required x and y integer columns, created by the v2 baseline revision f5ef4347cb2a. They hold hex-map positions.
Nothing needs these values: the app draws its UK maps from its own layout files, and nothing in the Public API reads or writes either table yet. No managed v2 database role holds any privilege on them (policyengine_api/data/v2/privileges.py).
Intended result
The two tables have no x or y columns, removed by an autogenerated v2 Alembic revision whose downgrade adds them back, and the SQLModel tables match.
Deployment
Deploying the revision changes the staging and production v2 schemas. The merge needs Anthony's approval and waits until after the UK Budget on 28 October 2026.
- Langage dominant
- Python
- Étoiles
- 18
- Forks
- 33
- Merge moyen
- 2 j 7 h
- PR mergées (30 j)
- 64
Préparer son environnement
- Aucun Dockerfile ni fichier Docker Compose
- Propose un modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de PolicyEngine/policyengine-api
-
Difficulté 2/5 1-3 heures Accessibilité débutants 76/100
PolicyEngine/policyengine-api#4037 ·
Les mainteneurs répondent en général sous 2 jours
-
Difficulté 1/5 1-3 heures Accessibilité débutants 88/100
PolicyEngine/policyengine-api#3911 ·
Les mainteneurs répondent en général sous 2 jours
-
Difficulté 1/5 Moins d'une heure Accessibilité débutants 82/100
PolicyEngine/policyengine-api#3910 ·
Les mainteneurs répondent en général sous 2 jours
-
Split the release tests over six runners, and stop repeating the merge run's checks on the release commitPeut-être pris @anth-volk l’a pris aujourd’hui. Ouverte
Difficulté 4/5 3-5 jours Accessibilité débutants 25/100
PolicyEngine/policyengine-api#4133 ·
Les mainteneurs répondent en général sous 2 jours
-
Difficulté 4/5 3-5 jours Accessibilité débutants 48/100
PolicyEngine/policyengine-api#4131 ·
Les mainteneurs répondent en général sous 2 jours
Toutes les issues de PolicyEngine/policyengine-api
Issues similaires
-
changelog investigate
Difficulté 2/5 1-3 heures Accessibilité débutants 67/100
ramnes/notion-sdk-py#433 ·
-
docs pkg:core
Difficulté 1/5 Moins d'une heure Accessibilité débutants 84/100
pydantic/pydantic-ai#11929 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
intake/intake-esm#789 ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 82/100
MemMachine/MemMachine#1877 ·
Les mainteneurs répondent en général sous 3 jours
-
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
openedx/openedx-authz#579 ·
Les mainteneurs répondent en général sous 1 jour