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

Remove a tabela balance_operations #1673

Merged
merged 1 commit into from
Apr 25, 2024
Merged

Conversation

Rafatcb
Copy link
Collaborator

@Rafatcb Rafatcb commented Apr 10, 2024

Mudanças realizadas

A migração de balance_operations para as três novas tabelas (content_tabcoin_operations, user_tabcoin_operations e user_tabcash_operations) foi concluída, vide #1661 (comment). Este PR remove a tabela balance_operations, que não é mais utilizada, e a função get_current_balance, que acessava a tabela.

Tipo de mudança

  • Refatoração

Checklist:

  • Os testes antigos estão passando localmente.

@Rafatcb Rafatcb added back Envolve modificações no backend refatoração Melhoria no código que não modifica o comportamento externo labels Apr 10, 2024
Copy link

vercel bot commented Apr 10, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
tabnews ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 24, 2024 4:38pm

@Rafatcb Rafatcb changed the title Drop balance operations Remove a tabela balance_operations Apr 10, 2024
@aprendendofelipe
Copy link
Collaborator

Já é seguro rodar essa migraton, não é @Rafatcb?

@Rafatcb
Copy link
Collaborator Author

Rafatcb commented Apr 19, 2024

Já é seguro rodar essa migraton, não é @Rafatcb?

Sim, não vi nenhuma anomalia ou erro ao longo desses dias.

@aprendendofelipe
Copy link
Collaborator

Rodei em homologação! 🚀

E os próximos deploys já estarão com a API funcionando normalmente, pois já voltei UNDER_MAINTENANCE para {"methodsAndPaths":[]}

Em produção não vai mudar nada que impacte o serviço, então podemos aguardar um pouco mais antes do merge. 🤝

Remove the `balance_operations` table, which was replaced by three tables about each specific type
of balance, and also remove the `get_current_balance` function, which accesses this table.
@aprendendofelipe aprendendofelipe force-pushed the drop-balance-operations branch from 5f262fa to dc4c487 Compare April 24, 2024 16:34
@aprendendofelipe aprendendofelipe merged commit 519a074 into main Apr 25, 2024
7 checks passed
@aprendendofelipe aprendendofelipe deleted the drop-balance-operations branch April 25, 2024 01:36
@aprendendofelipe
Copy link
Collaborator

Tabela removida com sucesso em produção! 🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
back Envolve modificações no backend refatoração Melhoria no código que não modifica o comportamento externo
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants