-
Notifications
You must be signed in to change notification settings - Fork 0
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
#492 [FIX]: Correção de manuseio de foto de perfil e navegabilidade após edição de perfil #493
#492 [FIX]: Correção de manuseio de foto de perfil e navegabilidade após edição de perfil #493
Conversation
…pós edição de perfil
…de criação de usuário
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.
- Maravilha! Deixei apenas algumas observações que talvez seja interessante discutir com o grupo.. Além disso, aproveitei para subir algumas alterações na lógica da seleção do papel do usuário: agora admin nunca deixará de ser ADMIN.
src/pages/CreateUserPage.jsx
Outdated
@@ -241,7 +241,7 @@ function CreateUserPage(props) { | |||
onPrimaryBtnClick: () => { | |||
if (response.data.data.id === user.id) | |||
renewUser(response.data.data.username, response.data.data.role, response.data.data.profileImage?.path); | |||
navigate(`/dash/institutions/my`); | |||
navigate(`/dash/profile`); |
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.
Ok! Mas se considerarmos o caso de um usuário atualizando o perfil de outro usuário, talvez o ideal fosse redirecionar para a página da instituição mesmo, não?!
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.
- Atualizado para redirecionar para página de instituições caso a ação não seja realizada sobre o próprio usuário;
src/pages/CreateUserPage.jsx
Outdated
}, | ||
}); | ||
} else { | ||
showAlert({ headerText: 'Usuário excluído com sucesso.', onPrimaryBtnClick: () => navigate(`/dash/applications`) }); |
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.
Se um usuário foi excluído e não foi o próprio usuário talvez o ideal seja voltar para a página da instituição a qual o usuário excluído pertencia ou então voltar para a página de instituições, não?
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.
- Atualizado para redirecionar para página de instituições caso a ação não seja realizada sobre o próprio usuário;
…a de criação de usuário
…il-e-redirecionamento-da-edicao-de-perfil
Esse pull request inclui várias alterações para melhorar o tratamento da imagem do perfil do usuário e a lógica de navegação no aplicativo. As alterações mais importantes incluem a modificação dos caminhos de origem da imagem, a adição de uma função de logout ao contexto do usuário e a atualização dos caminhos de navegação após as ações do usuário.
Aprimoramentos no tratamento da imagem do perfil do usuário:
src/components/Sidebar.jsx
: Atualizamos o caminho de origem da imagem do perfil para incluir "api/" para fins de consistência.src/pages/CreateUserPage.jsx
: Atualizamos o caminho de origem da imagem do perfil para incluir "api/" em vários locais para fins de consistência.src/pages/ProfilePage.jsx
: Atualizamos o caminho de origem da imagem do perfil para incluir "api/" para fins de consistência.Aprimoramentos na lógica de navegação:
src/pages/CreateUserPage.jsx
: Adição delogout
ao contexto do usuário e atualização dos caminhos de navegação após as ações do usuário para melhorar a experiência do usuário. [1] [2] [3]