Skip to content

[Mensageria IP] Adiciona novos campos no schema das tabelas para receber dados das novas listas#52

Merged
maaottoni merged 2 commits intomainfrom
feat/lista_nominal_mensageria
Feb 4, 2025
Merged

[Mensageria IP] Adiciona novos campos no schema das tabelas para receber dados das novas listas#52
maaottoni merged 2 commits intomainfrom
feat/lista_nominal_mensageria

Conversation

@waltmatheus
Copy link
Contributor

@waltmatheus waltmatheus commented Feb 3, 2025

Alterações da DDL das tabelas para receber dados de transmissão das novas listas de citopatológico, diabéticos e hipertensos

Motivo do ajuste:
Adaptar o schema das tabelas para adequar aos dados que serão transmitidos após a alteração do código fonte das listas nominais descritos nesse PR.

O que está sendo alterado:
Adição dos seguintes campos e seu tipo de formato:

Campo Tipo Descrição
co_fat_familia_territorio int64 Código da família do usuário
cidadao_telefone varchar(30) Número de telefone do usuário
cidadao_celular varchar(30) Número de celular do usuário
cidadao_situacao_trabalho varchar(500) Situação de trabalho do usuário
cidadao_povo_comunidade_tradicional varchar(500) Indica se pertence a uma comunidade tradicional
cidadao_identidade_genero varchar(500) Identidade de gênero do usuário
cidadao_raca_cor varchar(500) Raça/cor do usuário
cidadao_plano_saude_privado int4 Indica se possui plano de saúde privado
numero_visitas_ubs_ultimos_12_meses int4 Número de visitas à UBS nos últimos 12 meses

Próximos passos:

  • Comunicar ajustes importantes nos canais próprios de cada lista nominal
  • Atualizar documentação de regras de negócio no Notion

Summary by CodeRabbit

Summary by CodeRabbit

  • New Features

    • Foram adicionados novos campos para enriquecer as informações demográficas e de contato dos registros, incluindo dados como telefone, celular, identidade de gênero, raça, situação de trabalho e informações de visitas recentes.
  • Refactor

    • Atualizamos a exibição dos dados, alterando o indicador de plano de saúde para refletir a situação de trabalho, proporcionando uma visão mais precisa do contexto socioeconômico dos cidadãos.

@waltmatheus waltmatheus requested a review from maaottoni February 3, 2025 13:41
@coderabbitai
Copy link

coderabbitai bot commented Feb 3, 2025

Walkthrough

Foram realizadas alterações em scripts SQL que impactam a modelagem dos dados referentes a cidadãos diabéticos, citopatológicos e hipertensos. Em uma query, a coluna no SELECT foi renomeada de cidadao_plano_saude_privado para cidadao_situacao_trabalho. Além disso, novas colunas foram adicionadas aos esquemas das tabelas para incluir informações adicionais como telefone, celular, território, identidade de gênero, raça, entre outras, garantindo consistência e ampliando os dados demográficos armazenados.

Changes

Arquivo(s) Sumário das Alterações
transmissor_impulso_esus/codigos_listas_transmitidas/lista_nominal_diabeticos.sql Renomeação de coluna: cidadao_plano_saude_privadocidadao_situacao_trabalho no SELECT final.
transmissor_impulso_esus/ddls_tabelas_transmitidas/{citopatologico.sql, diabeticos.sql, hipertensos.sql} Adição de novas colunas: co_fat_familia_territorio, cidadao_telefone, cidadao_celular, cidadao_situacao_trabalho, cidadao_povo_comunidade_tradicional, cidadao_identidade_genero, cidadao_raca_cor, cidadao_plano_saude_privado, numero_visitas_ubs_ultimos_12_meses. Em hipertensos.sql, ajuste na formatação de municipio_id_sus.

Possibly related PRs

Suggested labels

enhancement

Suggested reviewers

  • helenafig

Poem

Sou um coelho saltitante a codificar,
Pelas linhas de SQL começo a dançar.
Renomeei e acrescentei com precisão,
Deixando os dados cheios de informação.
Que a tabela floresça como um jardim no ar! 🐇✨

Tip

🌐 Web search-backed reviews and chat
  • We have enabled web search-based reviews and chat for all users. This feature allows CodeRabbit to access the latest documentation and information on the web.
  • You can disable this feature by setting web_search: false in the knowledge_base settings.
  • Please share any feedback in the Discord discussion.

📜 Recent review details

Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between ef4cd8d and 5a01fd6.

📒 Files selected for processing (3)
  • transmissor_impulso_esus/ddls_tabelas_transmitidas/citopatologico.sql (2 hunks)
  • transmissor_impulso_esus/ddls_tabelas_transmitidas/diabeticos.sql (2 hunks)
  • transmissor_impulso_esus/ddls_tabelas_transmitidas/hipertensos.sql (2 hunks)
🚧 Files skipped from review as they are similar to previous changes (2)
  • transmissor_impulso_esus/ddls_tabelas_transmitidas/diabeticos.sql
  • transmissor_impulso_esus/ddls_tabelas_transmitidas/hipertensos.sql
🔇 Additional comments (2)
transmissor_impulso_esus/ddls_tabelas_transmitidas/citopatologico.sql (2)

5-6: Verificação de "municipio_id_sus" e novo campo "quadrimestre_atual".
A alteração do valor default de municipio_id_sus para '111111'::character varying está de acordo com a migração de tipo prevista, mas certifique-se de que essa mudança reflete corretamente a semântica desejada no modelo de dados. Além disso, a adição do campo quadrimestre_atual do tipo text deve estar alinhada com os requisitos do negócio, visto que esse campo não estava explicitamente citado na documentação dos novos campos.


36-44: Adição dos novos campos para captação dos dados estendidos.
Foram adicionadas as seguintes colunas:

  • co_fat_familia_territorio int4 NULL
  • cidadao_telefone varchar(30) NULL
  • cidadao_celular varchar(30) NULL
  • cidadao_situacao_trabalho varchar(500) NULL
  • cidadao_povo_comunidade_tradicional varchar(500) NULL
  • cidadao_identidade_genero varchar(500) NULL
  • cidadao_raca_cor varchar(500) NULL
  • cidadao_plano_saude_privado int4 NULL
  • numero_visitas_ubs_ultimos_12_meses int4 NULL

Recomendo revisar os seguintes pontos:

  • Tipo de co_fat_familia_territorio: A especificação PR menciona int64, mas aqui foi utilizado int4 (inteiro de 32 bits). Verifique se essa definição atende aos requisitos de amplitude dos dados ou se é necessário utilizar bigint.
  • Limites dos campos varchar: Confirme se os tamanhos de 30 e 500 caracteres são adequados para os dados reais, considerando possíveis formatos e padronizações dos números de telefone, identificação de gênero, entre outros.

Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?

❤️ Share
🪧 Tips

Chat

There are 3 ways to chat with CodeRabbit:

  • Review comments: Directly reply to a review comment made by CodeRabbit. Example:
    • I pushed a fix in commit <commit_id>, please review it.
    • Generate unit testing code for this file.
    • Open a follow-up GitHub issue for this discussion.
  • Files and specific lines of code (under the "Files changed" tab): Tag @coderabbitai in a new review comment at the desired location with your query. Examples:
    • @coderabbitai generate unit testing code for this file.
    • @coderabbitai modularize this function.
  • PR comments: Tag @coderabbitai in a new PR comment to ask questions about the PR branch. For the best results, please provide a very specific query, as very limited context is provided in this mode. Examples:
    • @coderabbitai gather interesting stats about this repository and render them as a table. Additionally, render a pie chart showing the language distribution in the codebase.
    • @coderabbitai read src/utils.ts and generate unit testing code.
    • @coderabbitai read the files in the src/scheduler package and generate a class diagram using mermaid and a README in the markdown format.
    • @coderabbitai help me debug CodeRabbit configuration file.

Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments.

CodeRabbit Commands (Invoked using PR comments)

  • @coderabbitai pause to pause the reviews on a PR.
  • @coderabbitai resume to resume the paused reviews.
  • @coderabbitai review to trigger an incremental review. This is useful when automatic reviews are disabled for the repository.
  • @coderabbitai full review to do a full review from scratch and review all the files again.
  • @coderabbitai summary to regenerate the summary of the PR.
  • @coderabbitai generate docstrings to generate docstrings for this PR. (Beta)
  • @coderabbitai resolve resolve all the CodeRabbit review comments.
  • @coderabbitai configuration to show the current CodeRabbit configuration for the repository.
  • @coderabbitai help to get help.

Other keywords and placeholders

  • Add @coderabbitai ignore anywhere in the PR description to prevent this PR from being reviewed.
  • Add @coderabbitai summary to generate the high-level summary at a specific location in the PR description.
  • Add @coderabbitai anywhere in the PR title to generate the title automatically.

CodeRabbit Configuration File (.coderabbit.yaml)

  • You can programmatically configure CodeRabbit by adding a .coderabbit.yaml file to the root of your repository.
  • Please see the configuration documentation for more information.
  • If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation: # yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json

Documentation and Community

  • Visit our Documentation for detailed information on how to use CodeRabbit.
  • Join our Discord Community to get help, request features, and share feedback.
  • Follow us on X/Twitter for updates and announcements.

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 0

🧹 Nitpick comments (1)
transmissor_impulso_esus/ddls_tabelas_transmitidas/hipertensos.sql (1)

31-31: Ajuste de formatação da coluna
Verifique a indentação da coluna profissional_nome_procedimento para garantir consistência com as demais definições de coluna.

📜 Review details

Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 9665893 and ef4cd8d.

📒 Files selected for processing (4)
  • transmissor_impulso_esus/codigos_listas_transmitidas/lista_nominal_diabeticos.sql (1 hunks)
  • transmissor_impulso_esus/ddls_tabelas_transmitidas/citopatologico.sql (1 hunks)
  • transmissor_impulso_esus/ddls_tabelas_transmitidas/diabeticos.sql (1 hunks)
  • transmissor_impulso_esus/ddls_tabelas_transmitidas/hipertensos.sql (1 hunks)
🔇 Additional comments (4)
transmissor_impulso_esus/ddls_tabelas_transmitidas/hipertensos.sql (1)

38-46: Novas colunas adicionadas - Validação de tipos e nomenclatura
As novas colunas (co_fat_familia_territorio, cidadao_telefone, cidadao_celular, cidadao_situacao_trabalho, cidadao_povo_comunidade_tradicional, cidadao_identidade_genero, cidadao_raca_cor, cidadao_plano_saude_privado e vu.numero_visitas_ubs_ultimos_12_meses) foram adicionadas para receber os dados dos novos campos. Verifique se os tipos utilizados (por exemplo, int64 e int4) atendem aos requisitos do projeto. Além disso, confirme se a nomenclatura com ponto em vu.numero_visitas_ubs_ultimos_12_meses é válida e intencional no seu ambiente, pois nomes de coluna com ponto podem exigir aspas para serem interpretados corretamente pelo SGBD.

transmissor_impulso_esus/ddls_tabelas_transmitidas/citopatologico.sql (1)

36-44: Novas colunas adicionadas na tabela de citopatológico
As novas colunas foram adicionadas de forma consistente com as alterações em outras tabelas. Contudo, assim como no arquivo de hipertensos, é importante confirmar se o uso do ponto na coluna vu.numero_visitas_ubs_ultimos_12_meses está de acordo com a sintaxe e as convenções do SGBD em uso.

transmissor_impulso_esus/ddls_tabelas_transmitidas/diabeticos.sql (1)

38-46: Novas colunas inseridas na tabela de diabéticos
Os novos campos foram adicionados corretamente para expandir o schema e acomodar os dados necessários. Recomenda-se verificar se os tipos int64 e int4 estão alinhados com os padrões esperados no projeto. Adicionalmente, valide se a nomenclatura vu.numero_visitas_ubs_ultimos_12_meses (com ponto) é suportada pelo SGBD ou se será necessário utilizar aspas.

transmissor_impulso_esus/codigos_listas_transmitidas/lista_nominal_diabeticos.sql (1)

337-341: Renomeação de coluna inconsistente no SELECT
Observou-se que o SELECT final retorna tanto cir.cidadao_situacao_trabalho (linha 337) quanto cir.cidadao_plano_saude_privado (linha 341). Conforme a mudança proposta, a coluna cidadao_plano_saude_privado deve ser renomeada para cidadao_situacao_trabalho para evitar redundância e inconsistências nos dados. Recomenda-se remover ou consolidar a coluna antiga, mantendo apenas a nova nomenclatura.

Exemplo de diff sugerido:

-    cir.cidadao_plano_saude_privado,

@waltmatheus waltmatheus self-assigned this Feb 3, 2025
@maaottoni maaottoni merged commit 1a0ef77 into main Feb 4, 2025
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.

2 participants