Skip to content
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

Feat/perfil do usuario #25

Merged
merged 13 commits into from
Jan 22, 2024
Merged

Feat/perfil do usuario #25

merged 13 commits into from
Jan 22, 2024

Conversation

rozbr96
Copy link
Contributor

@rozbr96 rozbr96 commented Jan 20, 2024

Essa PR aborda e resolve #5 e resolve #6

O usuário pode se cadastrar na plataforma, e, ao fazê-lo, já é associado aos modelos de perfil e de informações pessoais.
As informações pessoais são criadas vazias, para posterior edição pelo usuário.
O usuário pode editar suas informações, a partir da página de perfil.
Além disso, foram feitas algumas melhorias no front-end.

Prints das telas:
image
image
image
image
image
image

hreis1 and others added 6 commits January 19, 2024 17:30
Utiliza método after_create para vincular perfil com usuário pós cadastro

Co-authored-by: Gabriel Manika <[email protected]>
Organiza navbar

Co-authored-by: Gabriel Manika <[email protected]>
Foi invertido o "fluxo" de edição do perfil de usuário/informações
pessoais, fazendo com que a edição partisse do model profile, ao
invés do model personal_info

Co-authored-by: Caique Arruda <[email protected]>
Co-authored-by: Eliseu Ramos <[email protected]>
Co-authored-by: Danilo Martins <[email protected]>
rozbr96 and others added 5 commits January 22, 2024 10:29
Ao criar um perfil, as informações pessoais do usuário já devem estar presentes

Co-authored-by: anaresgalla <[email protected]>
Melhorias nas telas:
- visualização de dados pessoais
- edição de dados

Co-authored-by: anaresgalla <[email protected]>
Campo de visibilidade foi adicionado ao editar/visualizar perfil do
usuário

Co-authored-by: anaresgalla <[email protected]>
Co-authored-by: anaresgalla <[email protected]>
Co-authored-by: Caique Arruda <[email protected]>
@rozbr96 rozbr96 marked this pull request as ready for review January 22, 2024 14:56
Copy link

@akaninja akaninja left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Revisado. Deixei algumas questões para avaliarem.

app/models/user.rb Outdated Show resolved Hide resolved
spec/models/profile_spec.rb Outdated Show resolved Hide resolved
spec/models/user_spec.rb Outdated Show resolved Hide resolved
spec/system/visitor_logs_in_spec.rb Show resolved Hide resolved
spec/system/user_edits_personal_info_spec.rb Show resolved Hide resolved
spec/models/personal_info_spec.rb Show resolved Hide resolved
spec/system/user_views_own_profile_spec.rb Show resolved Hide resolved
@rozbr96 rozbr96 merged commit 2225553 into main Jan 22, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Perfil] Usuário regular cadastra informações pessoais [Perfil] Usuário regular cadastra resumo pessoal
5 participants