Feature/edicao de perfil #15
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
- Configura exibição de exceções dos testes
- incluir tradução do model Profile Co-authored-by: Lucas Aguilar <[email protected]> Co-authored-by: Moises Almeida <[email protected]>
Co-authored-by: Paulo Henrique Meneses <[email protected]> Co-authored-by: Lucas Aguilar <[email protected]>
- Adiciona mais traduções ao i18n - Ajusta o nome dos teste Co-authored-by: Lucas Aguilar <[email protected]> Co-authored-by: Moises Almeida <[email protected]>
- Adiciona mais traduções no i18n. - Alteração nas views profile (edit, show) - Adiciona teste de autenticação - Adiciona mensagem condicional para edição / preenchimente de perfil Co-authored-by: Lucas Aguilar <[email protected]> Co-authored-by: Moises Almeida <[email protected]>
- Refatora método #first_update? - Pendencia: Corrigir teste spec/system/user_edit_profile_spec.rb:36 Co-authored-by: Paulo Henrique Meneses <[email protected]> Co-authored-by: Moises Almeida <[email protected]>
- Separa informações do perfil em partial Co-authored-by: Paulo Henrique Meneses <[email protected]> Co-authored-by: Moises Almeida <[email protected]>
Co-authored-by: Paulo Henrique Meneses <[email protected]> Co-authored-by: Moises Almeida <[email protected]>
Co-authored-by: Paulo Henrique Meneses <[email protected]> Co-authored-by: Moises Almeida <[email protected]>
Co-authored-by: Paulo Henrique Meneses <[email protected] Co-authored-by: Moises Almeida <[email protected]>
Co-authored-by: Moises Almeida <[email protected]> Co-authored-by: Paulo Henrique Meneses <[email protected]>
- Adiciona tradução - Ajuste no teste de edição com sucesso Co-authored-by: Paulo Henrique Meneses <[email protected]> Co-authored-by: Lucas Aguilar <[email protected]>
3 tasks
…eature/edicao-de-perfil
Co-authored-by: Lucas Aguilar <[email protected]> Co-authored-by: Moises Almeida <[email protected]>
Co-authored-by: Lucas Aguilar <[email protected]> Co-authored-by: Moises Almeida <[email protected]>
|
Na resolução de conflitos de merge com a main, achamos um problema que permite o usuário ter mais de um perfil. Convertemos o PR para draft e vamos solucionar. |
OtavioLins
reviewed
Jan 19, 2024
There was a problem hiding this comment.
Bom pessoal, deixei alguns comentários. Uma coisa que me chamou atenção também é o front end da app. Vocês, como um time, pretendem trabalhar nisso depois? Caso sim, só tomem cuidado para não acabar deixando um backlog muito grande de futuras alterações no front
|
Eu comecei a revisão antes de vocês mudarem de volta pra Draft e terminei depois, daí minha revisão acabou indo assim mesmo. É bom que vocês já conseguem ver uns outros probleminhas, também :) |
- Resolve conflito no seeds.rb - Resolve conflito no routes.rb - Resolve no layouts/application.html.erb - Resolve conflito em views/profiles/edit.html.erb Co-authored-by: Lucas Aguilar <[email protected]> Co-authored-by: Moises Almeida <[email protected]>
- refatora Profile#full_name e testes unitários - refatora factory do Profile Co-authored-by: Moises Almeida <[email protected]> Co-authored-by: Lucas Aguilar <[email protected]>
Co-authored-by: Moises Almeida <[email protected]> Co-authored-by: Lucas Aguilar <[email protected]>
Co-authored-by: Moises Almeida <[email protected]> Co-authored-by: Lucas Aguilar <[email protected]>
Co-authored-by: Lucas Aguilar <[email protected]> Co-authored-by: Moises Almeida <[email protected]>
Co-authored-by: Lucas Aguilar <[email protected]> Co-authored-by> Moises Almeida <[email protected]>
Co-authored-by: Lucas Aguilar <[email protected]> Co-authored-by: Moisés Almeida <[email protected]>
|
Fizemos ajustes conforme as sugestões. Sobre o front-end. Incluimos classes do bootstrap mas ainda vamos conversar mais sobre o layout geral da aplicação |
OtavioLins
approved these changes
Jan 22, 2024
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.
Esse PR resolve a issue #3.
Adicionamos a funcionalidade de edição do perfil do usuário através da página meu perfil.
Não é obrigatório que o usuário preencha o perfil após o primeiro login.
As mensagens nas telas variam de acordo com o estado do perfil do usuário. Se não tem nenhum dado preenchido, as mensagens na tela o estimulam a preencher o perfil.
Perfil sem informação cadastrada.
Formulário do perfil sem informações cadastradas.
Perfil com informações já cadastradas.
Formulário do perfil com informações já cadastradas.
