[contrarazao] Substituindo as calls de swal e quill por calls dos wrappers criados no mapa #683
[contrarazao] Substituindo as calls de swal e quill por calls dos wrappers criados no mapa #683CaffeineIssues wants to merge 4 commits intodevelopfrom
Conversation
| swal.fire({ | ||
|
|
||
| function custom(title, type, html = null, showConfirmButton = false) { | ||
| return Swal.fire({ |
There was a problem hiding this comment.
Por que foi necessário colocar esses returns?
There was a problem hiding this comment.
return faz com que a função vire uma promise pra poder usar o .then(), foi até sugestão do @Junior-Shyko. De outra forma a função é void e retorna undefined ai não da pra usar.
There was a problem hiding this comment.
E nos outros lugares que estava usando do jeito antigo (sem promise), não vai quebrar?
There was a problem hiding this comment.
Quebra não. o McMessages só foi usado no recurso até agora e lá alguns casos também foi preciso adicionar o return. No geral o Swal já é uma promisse e o correto é que o wrapper seja uma promisse também, de outra forma o wrapper vai estar removendo funcionalidades da lib.
Esse texto deve ficar abaixo do título |
Ai é do wrapper que o @Junior-Shyko fez que posiciona o html nessa Área. Devo alterar o wrapper para que o html fique na parte de cima? mas aí vai alterar em todos que usarem o wrapper. |
|
Tem outros ajustes a serem feitos além desse aí. Então creio que esse PR não vai ser 'mergeado' agora |
2 participants
✅ Descrição do propósito desse Pull Request
No modulo de contra-razão Substituir calls de swal e quill por calls dos wrappers criados no mapa
🧭 Referência a Issue
#682
❓ O que foi feito para atingir isso?
Substituí as calls de swall e quill no modulo por calls dos wrappers McMessages e QuillEditor.
Fiz pequenos ajustes no McMessages para poder manter a logica da contrarazão e não precisar botar css relacionado ao quilleditor no modulo.
🏃♀️ Tipo de mudança
Marque as opções relevantes:
🕵️ Como foi testado?
Checklist: ✔️
Observação: