Skip to content

Correção na documentação do README.md, ajuste no arquivo config.py, add arquivo 2026-01-bb.ofx#114

Open
marcelomiky wants to merge 4 commits intomainfrom
jan2026
Open

Correção na documentação do README.md, ajuste no arquivo config.py, add arquivo 2026-01-bb.ofx#114
marcelomiky wants to merge 4 commits intomainfrom
jan2026

Conversation

@marcelomiky
Copy link

@marcelomiky marcelomiky commented Feb 9, 2026

corrige typo em README.md, alterei config.py para funcionar o beancount do extrato do BB, adicionei arquivo 2026-01-bb.ofx na pasta extratos

…nt do extrato do BB, adicionei arquivo 2026-01-bb.ofx na pasta extratos
@marcelomiky marcelomiky requested review from a team as code owners February 9, 2026 19:51
@gabubellon
Copy link

@marcelomiky provavelmente a gente não tem um CI/CD disso né ?
Tem alguma documentação adicional além to readme, tipo onde está o deploy ? Da pra por um todo de gerar sempre uma versão de validação dos PRs

Copy link

@gabubellon gabubellon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚢

@zsinx6
Copy link
Member

zsinx6 commented Feb 9, 2026

@gabubellon O deploy ta aqui: https://financeiro.python.org.br/

@gabubellon
Copy link

@zsinx6 acho que não me fiz claro. Em qual serviço está o deploy.

Tem alguns serviços (Heroku, Reply, etc) que sobem uma instância de validação quando têm um PR

Nesse caso ajudaria porque daria para validar se o load do ofx deu algum erro, porque é a parte mais chata do beancount.

@zsinx6
Copy link
Member

zsinx6 commented Feb 9, 2026

Acho que entendi, não tem nada de CI implementado, porém posso adicionar um (amanhã eu vou dar uma olhada em como poderia fazer validação de load desse arquivo)

@gabubellon
Copy link

@zsinx6 Só veja se tiver tempo, mas acho que vale sim ,

Posso ajudar com tempo, mexi um pouco com o beancount , da pra fazer alguns testes, e se eu me lembro da pra fazer consultas a "base de dados" , então daria pra com tempo evoluir em alguns testes sempre que se tem um arquivo novo

Copy link

@julianaklulo julianaklulo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Poderia colocar um título mais genérico no PR e editar a descrição?
Como a mensagem de commit era muito grande, ficou confuso com a quebra.

config.py Outdated
Comment on lines +4 to +5
ofx.Importer(r"78126\-6", "Assets:Bancos:BB", balance_type=ofx.BalanceType.NONE),
# ofx.Importer(r"78126\-6", "Assets:Bancos:BB", balance_type=ofx.BalanceType.NONE), # tive que alterar aqui para considerar os arquivos ofx que estava baixando do app do BB
ofx.Importer(r"^78126$", "Assets:Bancos:BB", balance_type=ofx.BalanceType.NONE),

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Deu algum problema na identificação do dígito da conta?
Acho que pode tirar o comentário e deixar isso documentado no corpo do PR

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Eu realmente não entendi pq ficou assim o arquivo ofx comigo. Vou ajustar no código com o seu comentário.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Vi algumas versões anteriores dos arquivos ofx, as de nov e dez de 2025 e estão com este dígito. No que baixei realmente não está.

@marcelomiky marcelomiky changed the title corrige typo em README.md, alterei config.py para funcionar o beancou… Correção na documentação do README.md, ajuste no arquivo config.py, add arquivo 2026-01-bb.ofx Feb 10, 2026
@marcelomiky
Copy link
Author

@zsinx6 Só veja se tiver tempo, mas acho que vale sim ,

Posso ajudar com tempo, mexi um pouco com o beancount , da pra fazer alguns testes, e se eu me lembro da pra fazer consultas a "base de dados" , então daria pra com tempo evoluir em alguns testes sempre que se tem um arquivo novo

@gabubellon, @zsinx6 e @julianaklulo, estou mapeando as etapas do fluxo de prestação de contas. Com isso, depois conto com vcs pra fazer uma força tarefa para analisar pontos de melhoria e implementamos. @talitarossari sugeriu algumas melhorias também pro fluxo da NF.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants