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

Conserta erro de lógica na tabela de profissionais #182

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

vitmesquita
Copy link
Contributor

Descrição & motivação

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).

Copy link

github-actions bot commented Jan 3, 2025

Modelos Modificados

Os seguintes modelos foram modificados:
- models/marts/core/dimensions/dim_profissional_saude.sql

Comando para rodar DBT

dbt run --select dim_profissional_saude+ --full-refresh

@vitmesquita
Copy link
Contributor Author

Join antigo descartava informações de profissionais
image

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