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

Inicio de sesión con SOLID #113

Merged
merged 5 commits into from
Mar 31, 2022
Merged

Inicio de sesión con SOLID #113

merged 5 commits into from
Mar 31, 2022

Conversation

mariaurrutia97
Copy link
Contributor

Hice el inicio de sesión con SOLID, a ver que os parece así🤔

@uo276406
Copy link
Contributor

uo276406 commented Mar 28, 2022

Yo más o menos lo que vi en clase me parecio que estaba bien. Lo único resolver los code smells para que no den problemas luego al desplegar que sean warnings... Pero bueno si es de frontend que lo revise mejor otro 😅

@mariaurrutia97
Copy link
Contributor Author

No me había dado cuenta de los code smells la verdad😅, ahora los resuelvo.

@sonarcloud
Copy link

sonarcloud bot commented Mar 29, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

Copy link
Contributor

@SofiaYYQ SofiaYYQ left a comment

Choose a reason for hiding this comment

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

A mí me funciona.
Una cosa que no funciona bien es que, cuando el usuario hace log out en la opción del menú y si se intenta volver a logearse no le pide contraseña🤔, ¿no debería haber salido de la sesión ya?

Como más cosas que se puede hacer con esta parte en el futuro, quizás, podemos hacer que permitiera a más proveedores de Solid POD.

@SofiaYYQ
Copy link
Contributor

A mí me funciona. Una cosa que no funciona bien es que, cuando el usuario hace log out en la opción del menú y si se intenta volver a logearse no le pide contraseña🤔, ¿no debería haber salido de la sesión ya?

Estuve mirando otras aplicaciones con Solid y les pasa lo mismo 😅

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
coding Coding implementations frontend
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Implementar sign in
4 participants