-
Notifications
You must be signed in to change notification settings - Fork 39
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
20187 - Disponibilizar os thrifts e xmls novos no esusab-integracao #120
base: master
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ajustar o circunferencia abdominal em alguns lugares.
Gerar alguns arquivos dnv com base no branch onde o circunferência abdominal está corrigido.
Remover alguns arquivos do pr que não alterou nada (não marquei todos)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Esse aqui da pra tirar do pr
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
tirar do pr
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
tirar do pr
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
tirar do pr
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
tirar do pr
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
gerar dnv
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
gerar dnv
@@ -108,3 +108,20 @@ struct ResultadosExameThrift { | |||
4:optional i64 dataResultado; | |||
6:optional list<ResultadoExameThrift> resultadoExame; | |||
} | |||
|
|||
struct MedicoesThrift { | |||
1:optional double circuferenciaAbdominal; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
1:optional double circuferenciaAbdominal; | |
1:optional double circunferenciaAbdominal; |
XSD/medicoes.xsd
Outdated
|
||
<xs:complexType name="medicoesTransport"> | ||
<xs:sequence> | ||
<xs:element minOccurs="0" name="circuferenciaAbdominal" type="xs:double"/> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
<xs:element minOccurs="0" name="circuferenciaAbdominal" type="xs:double"/> | |
<xs:element minOccurs="0" name="circunferenciaAbdominal" type="xs:double"/> |
@@ -136,6 +134,21 @@ | |||
<hipoteseDiagnosticoCid10>O03</hipoteseDiagnosticoCid10> | |||
<classificacaoRisco>1</classificacaoRisco> | |||
</encaminhamentos> | |||
<medicoes> | |||
<circuferenciaAbdominal>50.0</circuferenciaAbdominal> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
<circuferenciaAbdominal>50.0</circuferenciaAbdominal> | |
<circunferenciaAbdominal>50.0</circunferenciaAbdominal> |
Ajustar o circunferencia abdominal nos exemplos
Retirei alguns arquivos do PR e fiz alguns ajustes com base nos comentários do Wag. A geração dos arquivos é feita através de um .sh que existe no repositório esusab-integracao, então não faria diferença regerar após o versionamento. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Acho que essa não muda depois da https://github.com/laboratoriobridge/pec/issues/20950 né
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ela deve ter os campos depreciados sim, o problema da issue é que um atendimento individual estava enviando as medições tanto na FAI quanto na FP, sendo que deveria ser só na FAI
Se for um atendimento de procedimentos com o CBO que só preenche a FP, as medições devem ir na FP daí ;)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
💯 Esse deu trabalho kkkkkk
@wagnerbsantos, acho que vale ver com o @nathanmolinari se ele vai revisar ainda ou deixamos travado no teste |
8:optional double pesoAcompanhamentoNutricional; | ||
9:optional double alturaAcompanhamentoNutricional; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Acho que faltou arrumar os números no thrift, confirma @nathanmolinari ? Ou mantemos assim porque o CN ainda mapeia com esses números?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Depois de arrumar os número, tem que regerar os Thrift.java
, tá?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pelo o que olhei ele mapeia por esses números, tanto que tem alguns campos que pulam números, por campos que já foram depreciados, então não precisaria corrigir
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Após confirmação com o Nathan, não precisa corrigir a ordenação, esse mapeamento ainda vai valer
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ela deve ter os campos depreciados sim, o problema da issue é que um atendimento individual estava enviando as medições tanto na FAI quanto na FP, sendo que deveria ser só na FAI
Se for um atendimento de procedimentos com o CBO que só preenche a FP, as medições devem ir na FP daí ;)
closes laboratoriobridge/pec#20187