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

Feat/farmacia report gerencial #79

Merged
merged 45 commits into from
Aug 26, 2024
Merged

Conversation

ThiagoTrabach
Copy link
Contributor

@ThiagoTrabach ThiagoTrabach commented Aug 26, 2024

Descrição & motivação

Este PR cria o report de medicamentos com estoque critico.

Issues Relacionadas

Tipo de Mudança

  • Novo modelo
  • Correção de bug
  • Refatoração
  • Mudança que quebra compatibilidade
  • Documentação
  • Atualização/Instalação de dependências

A fazer antes do merge

Em caso de quebra de compatibilidade

  • Validar em dev o funcionamento correto dos reports, dashboard e modelos afetados

DAG de Linhagem:

Validação dos modelos:

Considerações de Impacto:

Mudanças nos modelos existentes:

Lista de Verificação:

  • Meu pull request representa uma peça lógica de trabalho.
  • Meus commits estão relacionados ao pull request e parecem limpos.
  • Meu SQL segue o guia de estilo da SMS-Rio.
  • dbt build completa com sucesso e os testes dbt passam (se não, detalhe por quê)
  • Materializei meus modelos apropriadamente (tipo de materialização e no dataset correto).
  • Adicionei testes e documentação apropriados a quaisquer modelos novos.
  • Adicionei policy tags apropriadas a quaisquer modelos novos.
  • Adicionei tags apropriadas a quaisquer modelos novos.
  • Atualizei o arquivo README (quando necessário).

…sistent naming convention for alias and update date filtering logic
…s__itens_com_movimento.sql, mart_estoque__report_medicamentos_controlados__itens_sem_movimento.sql, and mart_estoque__report_medicamentos_controlados__relacao_relatorios.sql
…ion fix and farmacia_popular_disponibilidade_indicador column
@ThiagoTrabach ThiagoTrabach merged commit 7f46c3f into master Aug 26, 2024
3 of 7 checks passed
@ThiagoTrabach ThiagoTrabach deleted the feat/farmacia-report-gerencial branch August 26, 2024 18:23
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.

1 participant