fix(mockup): BUG-400b — remove non-existent columns from saveMockupToDb INSERT#571
Conversation
…Db INSERT position_x, position_y, logo_width_cm, logo_height_cm do not exist as columns in generated_mockups. They were silently rejected by PostgREST on every mockup save. The values are already preserved inside area_config JSONB (positionX, positionY, logoWidth, logoHeight), so no data is lost. Also updated header comment to document all BUG-400 fixes applied.
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Warning Review limit reached
More reviews will be available in 34 minutes and 22 seconds. Learn how PR review limits work. Your organization has run out of usage credits. Purchase more in the billing tab. ⌛ How to resolve this issue?After more reviews become available, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans include higher PR review limits than trial, open-source, and free plans. In all cases, reviews become available again over time. During sustained high-volume PR review activity, CodeRabbit may temporarily slow when the next review becomes available. Please see our Fair Usage Limits Policy for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
|
This pull request has been ignored for the connected project Preview Branches by Supabase. |
Bug
saveMockupToDbtentava inserir 4 colunas que não existem emgenerated_mockups:O insert não falhava visivelmente porque o código tem
try/catchque silencia erros — mas o Supabase rejeitava essas colunas em cada salvamento de mockup.Por que não há perda de dados
Esses valores já estavam sendo salvos corretamente dentro de
area_configJSONB (que foi adicionado nessa mesma função):E o
fetchMockupHistory(corrigido no PR #570) já extrai esses valores dearea_configpara expor comoposition_x,position_y, etc.Fix
Remover as 4 linhas duplicadas do INSERT. 1 arquivo, 4 linhas removidas.
Summary by cubic
Removed non-existent columns position_x, position_y, logo_width_cm, and logo_height_cm from saveMockupToDb INSERT to resolve Linear BUG-400b. This stops silent PostgREST rejections on save; the values are already saved in area_config JSONB, so no data loss.
Written for commit 748ab8e. Summary will update on new commits.