Repository navigation
chore(release): integra a dev na main — dados bancários e PIX editáveis no colaborador (#1041) - #1046
Merged
Merged
Conversation
…r no histórico Os colaboradores migrados do legado vieram sem dados bancários, porque o legado não tinha esse campo, e dado bancário muda. O PUT /collaborators/:id passa a aceitar bankAccount/pixKey para incluir e trocar, com a mesma validação da criação e a permissão collaborator:write. - Semântica própria: ausente ou null MANTÉM o gravado; objeto valida e substitui. O front em produção envia null em todo PUT, então null = remover apagaria os dados a cada edição. Diverge do PUT do Fornecedor de propósito, documentado no schema. - Histórico: o texto do banco passa a incluir o DV (troca só do DV agora gera linha) e o rótulo vira "Dados bancários". - Autor: cada linha da edição grava id e nome do usuário no momento da ação (migration 0021, colunas nullable). Nome resolvido pelo auth/public-api; se indisponível, grava o id com nome nulo. O CSV de histórico ganha a coluna alterado_por no fim. Autor na inativação/reativação fica para a #1040. Closes #1029 Assisted-by: Claude-Code:claude-opus-5-5
- O port de leitura do nome do autor é aberto no server.ts com a connection string do auth (não a do partners) e degrada se falhar, como o de programs: o boot não cai por uma leitura opcional. - A leitura do nome tem timeout de 2 s, porque roda depois do save, e nome vazio vira null (uma só representação de "sem nome"). - O histórico do PIX inclui o tipo da chave (tipo:chave): trocar só o tipo agora gera linha, o mesmo defeito que o DV tinha no banco. - Os campos de banco/PIX ganham .max() igual às colunas: valor grande demais é 400 na borda, e não erro de servidor no UPDATE. - Os dois helpers de "manter ou validar" viram um só, genérico. Refs #1029 Assisted-by: Claude-Code:claude-opus-5-5
…os-bancarios feat(partners): editar dados bancários e PIX do colaborador, com autor no histórico
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
O que entra
Integração da
devnamain. São 3 commits, todos da edição de dados bancários e PIX do colaborador (PR #1041, mergeado nadev, fecha a #1029):fe2b9aac: oPUT /collaborators/:idaceitabankAccountepixKey, e o histórico passa a gravar o autore76a07d8: correções da revisão (timeout na leitura do nome do autor,tipo:chaveno histórico do PIX,.max()na borda)996bfe0f: merge da feat(partners): editar dados bancários e PIX do colaborador, com autor no histórico #1041Validado pela P.O. no ambiente local e na homologação, junto com o front (web-app#417, spec 120). Isso inclui o CSV do histórico registrando a alteração.
Este PR tem de entrar na
mainantes do front da spec 120. O web-app envia banco e PIX noPUT; amainatual descarta os dois campos sem erro, e a tela pareceria ter salvado. O PRdevelop → maindo web-app só será aberto depois deste merge.O que muda em produção
0021(partners)par_collaborator_history+changed_by_user_id varchar(36)echanged_by_name varchar(128), as duas anuláveis. SóADD COLUMN, sem backfillPUT /collaborators/:idnullmantém o banco/PIX gravado (retrocompatível com o front em produção, que mandanullem todoPUT); objeto valida e substituicollaborator:write, a mesma da ediçãoalterado_porno fim; as 9 colunas anteriores não se movemserver.tsauthcommodules.auth.connectionStringe o injeta nopartners. Se a abertura falhar, degrada: grava o histórico sem o nome do autor e não derruba o boot27 arquivos, 2.398 inserções, 33 remoções. Nenhuma variável de ambiente nova.
Observação sobre a topologia
O único commit que a
maintem a mais é o merge commit da promoção anterior (#1031), cujo conteúdo já está nadev. Não há diferença de conteúdo na direçãomain → dev.🤖 Generated with Claude Code