Feature/criar conta #12
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Co-authored-by: Adoniran France <adoniranfranceh@users.noreply.github.com>
Co-authored-by: Adoniran França <adoniranfranca@gmail.com>
Co-authored-by: Adoniran França <adoniranfranca@gmail.com>
Co-authored-by: Adoniran France <adoniranfranceh@users.noreply.github.com>
Co-authored-by: Adoniran França <adoniranfranca@gmail.com>
…usuário Co-authored-by: Adoniran França <adoniranfranca@gmail.com>
Co-authored-by: Adoniran França <adoniranfranca@gmail.com>
5 tasks
|
Parece que faltou marcar qual issue esse PR resolve. |
Perdão, não vi que estava linkado do lado. |
|
Mas é uma boa adicionar no corpo a issue também, acredito. Vou editar no corpo! |
OtavioLins
reviewed
Jan 18, 2024
There was a problem hiding this comment.
Bom trabalho pessoal, mas comentei algumas coisas que precisam de refinamento. Além disso, é sempre importante colocar prints nos PR onde trabalhamos com atributos visuais, para que os revisores e seus colegas de time possam ver como as telas estão e como os erros estão aparecendo :)
Remove o helper de profile que não estava sendo usado. Adiciona teste de unidade para garantir a criação de um perfil vazio. Adiciona testes de sistema para visualização das mensagens de erro na criação de conta. Remove testes de sistema redundantes de visualização de formulários. Co-authored-by: Adoniran France <adoniranfranca@gmail.com>
Co-authored-by: Adoniran França <adoniranfranca@gmail.com>
|
Revisões implementadas @OtavioLins, aguardando comentários para resolver os conflitos. |
Co-authored-by: Adoniran França <adoniranfranca@gmail.com>
Co-authored-by: Adoniran França <adoniranfranca@gmail.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Este pull request adiciona a funcionalidade de cadastro de usuário na aplicação e também garante o fluxo completo de login e logout, conforme descrito na issue #2
Para um cadastro bem sucedido o usuário precisa informar um CPF válido, um e-mail e uma senha com confirmação. Tanto CPF quanto e-mail devem ser únicos na plataforma.
Para validação de CPF utilizamos a gem cpf_cnpj.
A função de redirecionamento para a edição de perfil após a criação bem sucedida de conta depende das rotas de
profile. Portanto, para não ficarmos com o débito deste critério de aceite criamos um controller de perfil com uma única action deedit, além de sua rota e view. Assim conseguimos implementar o redirecionamento para a página correta. Não adicionamos nenhum HTML na viewedit, deixamos o HTML padrão criado pelo Rails.Telas
Tela de cadastro
Tela de cadastro com erros de campos vazios
Tela de cadastro com erros de campos inválidos
Tela de login
Tela de login com campos inválidos