-
Notifications
You must be signed in to change notification settings - Fork 15
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
Enhancement/forms #658
Enhancement/forms #658
Conversation
src/elm/Page/FormPlayground.elm
Outdated
GotUserFormMsg subMsg -> | ||
Form.update subMsg model.userFormModel | ||
|> UR.fromChild | ||
(\userFormModel -> { model | userFormModel = userFormModel }) | ||
GotUserFormMsg | ||
SubmittedUser | ||
model |
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.
@lucca65 this is what I was talking about having a component using UpdateResult
. In this case, whenever a valid User
is submitted, the SubmittedUser
message is fired, and then the parent (this page) can do whatever it wants with the user
Shop Buy (versão mobile)
Não entendi qual seria esse fluxo: |
Resolvido ✅
Não consegui replicar, para mim aparece normal:
Como esse não é o foco desse PR, apenas ajustei alguns detalhes mais simples, mas sugiro revisitarmos essa tela em outro PR
Não consegui replicar, os toggles funcionam normalmente
Isso é intencional - a comunidade deve falar conosco para habilitar essa funcionalidade (idem para Sponsorship)
Isso acontece porque o toggle está desabilitado (não aceita input) - o mesmo acontece com o KYC, pois esses dois itens devem ser ativados/desativados por nós, manualmente. Talvez possamos mudar o estilo do toggle quando ele está desabilitado, o que acha?
Resolvido ✅
Quando estávamos criando o sponsorship, concordamos em deixar assim por enquanto, e se os admins quiserem mudar as configurações, eles devem falar conosco
Mesma coisa de cima ☝️
Resolvido ✅
Resolvido ✅ coloquei os campos um em cima o outro em telas menores
Na minha opinião, o fluxo de mostrar na tela de notícias é melhor:
Resolvido ✅
Resolvido ✅
Resolvido ✅
Esse é o modal de adicionar contato que aparece quando o usuário não tem nenhum contato cadastrado. Na época que implementamos esse modal, decidimos que ele só deveria aparecer por um tempo limitado (até 01/01/2022), então ele não aparece mais. @lucca65 @juramos-2020 acham que devemos mostrar ele ainda ou podemos remover do código? |
Resolvido ✅ Coloquei um pouco mais de espaço, mas já vamos revisitar essa tela no shop facelift (#657)
Esse é o comportamento da tela: o usuário seleciona quantas unidades deseja comprar, e o campo de valor é automaticamente atualizado para mostrar o quanto vai custar a quantidade desejada. Novamente, podemos revisitar melhor esse fluxo no shop facelift |
Temos a #670 que pode ser um bom lugar para resolver e melhorar essa implementação de ações |
concordo. @juramos-2020 oq acha de melhorarmos essa comunicação? colocar um elemento para explicar melhor essas situações? Pode criar uma issue pra isso @juramos-2020? acho que não vai ser complexo de melhorar e vai dar bons resultados |
É verdade, eu tinha esquecido que tinha data pra ele deixar de aparecer. acho que é útil e deve voltar a existir, ou deveria no mínimo ser incluído diretamente no cadastro, pois muitos usuários ficam incomunicáveis Mas também é assunto para outra issue pessoal, acho que por hora podemos colocar o prazo do modal pra essa sexta pra gente testar os inputs e fluxo. assim testamos e tratamos melhor em separado depois. a próxima versão não sai essa semana então está tranquilo |
Obrigada @NeoVier pelos ajustes ficou show ! Sobre o modal do PIN (que aparece ao tentar salvar uma edição do objetivo) e os toogles, eu não consigo te mostrar visualmente porque a guia anônima não permite gravar e nem printar a tela. Mas pelo menos fica o meu registro aqui de como ocorreu o teste 😉👍👍
Tranquilo em relação a isso, o que eu quis dizer é que quando habilitamos um toogle a palavra muda de cor.
sobre isso, eu quis dizer com |
+1 |
Pensei e colocar o mesmo elemento que tem no KYC, aquele "?" com uma mensagem: Para ativar esse recurso a pessoa que administra a comunidade deve entrar em contato com equipe da Cambiatus para solicitar essa ativação. O que vocês acham? |
@lucca65 @NeoVier Nova issue criada 😉👍 |
@lucca65 @juramos-2020 aumentei a data limite do modal para essa sexta, agora é só acessar o dashboard sem nenhum contato cadastrado que ele aparece.
Entendi. Resolvido ✅ |
Consegui testar nas duas versões, tudo certo 😉👍 |
@lucca65 então acho que estamos prontos para mergear! Só preciso de uma aprovação no PR |
What issue does this PR close
Closes #585, closes #567
Changes Proposed ( a list of new changes introduced by this PR)
Form
module, responsible for all forms in our app. It's pretty complex, but it's pretty well documented, and there's also more documentation on hecrj/composable-form, which was a great inspiration for this module, so if you want to understand how it all works and how to use it, check those out!Form.*
modules, such asForm.Text
. These are responsible for each kind of field we can have in our app. They're pretty much the same as the previousView.Form.*
modules, but refactored to work with the newForm
moduleUR.addChild
andUR.fromChild
helpers. With these, we can have components that useUpdateResult
in theirupdate
function, and have a better API for getting results from these components (Form
is one example that already usesUpdateResult
on it'supdate
function).View.Form.*
modules. These shouldn't be needed anymore with the new form field modules.How to test ( a list of instructions on how to test this PR)
Since this changes every single form in our app, it's a good idea to run through the app and give it a complete test anywhere there is a form, checking if everything works. Here are some things that are trickier and might require some more attention:
Also, now we have something called elm-book! It's a visual guide to all our components (only form components are included for now), and you can access it at https://cambiatus-elm-book.netlify.app/, or on the same area where you can find our regular netlify preview of the app (scroll all the way down this page, near where you can type a comment). Make sure the language used and all of the examples are coherent. You can use those examples to more easily test our form components!