Hacktoberfest 2026: as issues que os mantenedores marcaram para outubro, abertas e boas para iniciantes. Ver issues do Hacktoberfest

Drop the unused x and y columns from API v2's UK area impact tables

Aberta
  • Algo errado?
#4,102 0 comentários 0 reações 0 responsáveis Ver no GitHub

Mantenedores costumam responder em até 2 dias

@anth-volk já está trabalhando nisso.

Desde 28/10/2026.

  • #4103 de @anth-volk — aberto

Avaliação

Dificuldade
2/5
Tempo estimado
1-3 horas
Facilidade para iniciantes
35/100
Tipo de issue
Refatoração
Clareza
Claramente especificada
Status de atividade
Estagnada
Stack de tecnologia
python, sqlalchemy
Domínio
api, database

Direção de pesquisa

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.

Escrita pelo modelo de indexação a partir do texto da issue.

Descrição

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.

Linguagem predominante
Python
Estrelas
18
Forks
33
Merge médio
2d 13h
PRs com merge (30d)
56

Preparar o ambiente

Primeiros passos

  1. Leia a issue inteira e depois o guia de contribuição do projeto.
  2. Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
  3. Faça um fork do repositório e trabalhe em uma branch.
  4. Abra um pull request que referencie o número da issue.

Mais de PolicyEngine/policyengine-api

Todas as issues de PolicyEngine/policyengine-api

Issues semelhantes

Mais issues de Python

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.