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.

Sign up for GitHub

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

Jump to bottom

PO-Upload - Parâmetros do payload #1395

Open
wsteixeira opened this issue Sep 13, 2022 · 6 comments · May be fixed by #1404
Open

PO-Upload - Parâmetros do payload #1395

wsteixeira opened this issue Sep 13, 2022 · 6 comments · May be fixed by #1404
Labels
enhancement New feature or request status: triage Issue or pull request that need to be triaged and assigned to a reviewer

Comments

@wsteixeira
Copy link
Contributor

Descrição da melhoria

Estou trabalhando em um projeto de integração com o TAE(Totvs Assinatura Eletrônica) e preciso fazer um upload de arquivos, porem o payload gerado é incompatível com a API da aplicação.

Caso de uso

Payload esperado:
image

Payload gerado pelo po-upload:
image

Informações adicionais

Seria interessante que no evento p-upload tivesse um atributo booleano para informar se o objeto data deve ser desestruturado ou não, ou criar um objeto extra se seria automaticamente desestruturado, assim seria possível passar parâmetros no mesmo nível do parâmetro `fileList'
@wsteixeira wsteixeira added the enhancement New feature or request label Sep 13, 2022
@wsteixeira wsteixeira changed the title PO-Upload - Paramentros do payload PO-Upload - Parâmetros do payload Sep 13, 2022
@jkozima
Copy link
Collaborator

jkozima commented Sep 13, 2022

Wanderley, Boa Tarde,
Obrigado por reportar, vamos avaliar a sua solicitação em conjunto com a PO.

Abraço,

@alinelariguet alinelariguet added the status: triage Issue or pull request that need to be triaged and assigned to a reviewer label Sep 13, 2022
@wsteixeira
Copy link
Contributor Author

Olá @jkozima e @alinelariguet,

Eu posso fazer uma contribuição, mas preciso saber qual solução seria mais adequada:

  • Atributo booleano para desestruturar o objeto data;
  • Parâmetro extra com desestruturação automática;
  • Outra solução;

@wsteixeira
Copy link
Contributor Author

Olá,

Na verdade minha sugestão de encaixa mais em uma extração de atributos do objeto data ou de um novo objeto, objetivamente seria criar uma interação para extrair os atributos do objeto e deixar no mesmo nível do fileList no payload.

@alinelariguet
Copy link
Member

Bom dia @wsteixeira, nesse caso seria uma quebra para quem já utiliza hoje certo? Vou dar uma olhada no padrão de API's e logo darei um retorno.

@wsteixeira
Copy link
Contributor Author

Olá @alinelariguet,

Eu acredito que não quebra, porque a ideia seria acrescentar no objeto gerado pelo evento do (p-upload)="uploadEvent($event)" uma das duas opções:

  • Opção 1: Acrescentar um atributo booleano opcional por exemplo extractData, que se informado e for igual a true, será feito a extração dos atributos do objeto data e manterá o objeto data vazio;
  • Opção 2: Acrescenta uma tratativa para um objeto adicional e opcional por exemplo extraParams, que se informado, será feito a extração automática dos atributos do mesmo.

As APIs do TAE estão no link https://totvs-sign.readme.io.

wsteixeira added a commit to wsteixeira/po-angular that referenced this issue Sep 21, 2022
@wsteixeira
feat(upload): adiciona o parâmetro extractData no evento p-upload
6895082 
Adiciona a propriedade `extractData` no evento `p-upload` e  nesta
propriedade pode ser informado algum dado solicitado pela API que
não possa estar no objeto `data`, assim o conteúdo sará extraído do
objeto e enviado como parâmetro.

Fixes po-ui#1395
wsteixeira added a commit to wsteixeira/po-angular that referenced this issue Sep 21, 2022
@wsteixeira
feat(upload): adiciona o parâmetro extractData no evento p-upload
ae33c8f 
Adiciona a propriedade `extractData` no evento `p-upload` e  nesta
propriedade pode ser informado algum dado solicitado pela API que
não possa estar no objeto `data`, assim o conteúdo será extraído do
objeto e enviado como parâmetro.

Fixes po-ui#1395
@wsteixeira wsteixeira linked a pull request Sep 21, 2022 that will close this issue
Open
6 tasks
wsteixeira added a commit to wsteixeira/po-angular that referenced this issue Sep 21, 2022
@wsteixeira
feat(upload): adiciona o parâmetro extractData no evento p-upload
756b641 
Adiciona a propriedade `extractData` no evento `p-upload` e  nesta
propriedade pode ser informado algum dado solicitado pela API que
não possa estar no objeto `data`, assim o conteúdo será extraído do
objeto e enviado como parâmetro.

Fixes po-ui#1395
@wsteixeira
Copy link
Contributor Author

Olá @alinelariguet,

Eu me antecipei e fiz uma contribuição, caso não seja a melhor saída ,por favor me informe.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request status: triage Issue or pull request that need to be triaged and assigned to a reviewer
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat(upload): adiciona o parâmetro extractData no evento p-upload wsteixeira/po-angular
3 participants
@wsteixeira @alinelariguet @jkozima