[CLKF-87] Implementa controle de locale no package verify#151
[CLKF-87] Implementa controle de locale no package verify#151bob-clk wants to merge 17 commits intofeature/CLKF-54from
Conversation
| this.listen = {}; | ||
| this.endpoint = 'https://app-workspaces-1.clicksign.dev/identity_authenticator'; | ||
| this.locale = ''; | ||
| this.endpoint = 'https://app.clicksign.com'; |
There was a problem hiding this comment.
Poderíamos usar a URL do app-workspaces de production:
| this.endpoint = 'https://app.clicksign.com'; | |
| this.endpoint = 'https://app.clicksign.com/app/verify/'; |
PS: Ainda vai ser necessário alterar lá no app-workspaces o namespace pra verify
There was a problem hiding this comment.
O endpoint é configurável a nível do cliente. Não seria mais viável manipular isso diretamente na lógica do path como foi feito?
:link
embedded/packages/verify/src/embedded.js
Lines 63 to 67 in 5c609d7
Ficando diretamente assim:
get path() {
if (this.locale) return `app/verify/${this.locale}/transactions/${this.key}`;
return `app/verify/sessions/${this.key}`;
} - Seguindo o padrão do widget embedded, o endpoint está apontando direto para o "host" (ambiente) sem path. Quem já utilizar outro embedded pode se guiar pelo padrão de instalação/configuração.
- Em caso de mudanças de paths é masi fácil disponibilizar uma atualização do script (mantendo ou não um suporte com redirect pro path legado) ou solicitar e aguardar que os clientes atualizem suas configurações?
Eu acredito que é mais prático o controle do nosos lado.
Exemplo (hipotético e exagerado) que o cliente externo precisaria alterar em caso de mudanças internas de roteamento:
this.endpoint = 'https://app.clicksign.com/outro-prefixo/outro-path/subpath/...';0ce4061 to
6348876
Compare
7440ed6 to
e88eb33
Compare
| @@ -0,0 +1,285 @@ | |||
| lockfileVersion: '9.0' | |||
There was a problem hiding this comment.
Esse arquivo não deveria ser inserido, estamos usando o yarn no projeto (por enquanto).
Ter 2 "package".lock trás uma "crise de identidade" e pode gerar alguns problemas a nível local como incosistencia de versões.
O gerenciamento da pasta node_modules são totalment diferentes entre as ferramentas: yarn (flattening) e pnpm (symlinks). E em caso de alternar entre os dois pode corromper a pasta de instalações, forçando a remoção total da pasta/reinstalação constantemente.
Acredito que não seja nosso caso, mas também pode gerar problemas também em CI: problema com detecção automática do instalador, duplicidade de build (se processar ambos), invalidação de cache desnecessário e etc.
There was a problem hiding this comment.
Caso queira executar a tarefa de migração, devido a complexidade do projeto e principalmente no uso em CI. Indico fazer em um pull request à parte.
Descrição
package/verifyidentity-authenticatorparaverifyClicksignVerifyverifyno path do endpointpackage/widgetv2parawidgetpackage/widget/v1v1paralegacye move parapackage/widgetIssue tracker
CLKF-87
Screenshots (para mudanças de UI, se houver)
Links e observações
Checklist para poder mergear