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(upload): inclui a propriedade url no evento upload #1835

Merged
merged 1 commit into from
Oct 6, 2023

Conversation

wsteixeira
Copy link
Contributor

@wsteixeira wsteixeira commented Oct 1, 2023

Esta melhoria permite a alteração da url no evento upload, assim o componente fica compativel com a API de upload do Fluig.

Fixes #1825

Upload

1825


PR Checklist [Revisor]

  • Padrão de Commit (Coeso, de acordo com o que está sendo realizado)
  • Código (Boas práticas, nome de variavéis/métodos, etc.)
  • Testes unitários (Cobre a situação implementada e coverage está mantido)
  • Documentação (Clara, objetiva e com exemplos caso necessário)
  • Samples (A implementação possui exemplo no Labs/Caso de uso)
  • Rodado em navegadores suportados (Chrome, FireFox, Edge)

Qual o comportamento atual?
O componente po-upload não é compativel com a API de upload do Fluig, pois o nome do arquivo em upload precisa ser informado na url para realizar o upload pela API do Fluig.

Qual o novo comportamento?
Com a melhoria o componente permite a alteração da url no evento upload do componete e isso possibilita a inclusão do nome do arquivo na url.

Simulação
Para realizar a simulação pode ser usado este App.zip.

image
image

@anliben
Copy link
Contributor

anliben commented Oct 2, 2023

Bom Dia @wsteixeira , tudo bem?

Obrigado pela contribuição, em breve iremos analisar e retornar uma resposta.

Attr.

jnrpalma
jnrpalma previously approved these changes Oct 2, 2023
Esta melhoria permite a alteração da `url` e `headers` no evento
`upload`, assim  o componente fica compativel com a API de upload
do Fluig conforme abaixo:

`url` - Será atualizada para conter o nome do arquivo enviado;
`headers` - No caso de autenticação `OAuth`, será gerado um nova
`Authorization`;

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

@jnrpalma, boa tarde,

Me desculpe, pois você já tinha aprovado, mas no caso de autenticação OAuth eu precisei incluir também o headers no evento, pois em caso de alteração da url, será necessario gerar uma nova Authorization.

Att.

@jnrpalma
Copy link
Contributor

jnrpalma commented Oct 3, 2023

Boa tarde @wsteixeira tudo bem !?
Sera revisado novamente grato !

@rafaellmarques rafaellmarques merged commit 6c8a9af into po-ui:master Oct 6, 2023
7 checks passed
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.

Upload: Componente incompatível com o Fluig
5 participants