Skip to content
This repository has been archived by the owner on Jun 10, 2024. It is now read-only.

PR/92-errores-varios #93

Merged
merged 3 commits into from
Mar 29, 2024
Merged

PR/92-errores-varios #93

merged 3 commits into from
Mar 29, 2024

Conversation

fravilpae
Copy link
Contributor

En esta PR hemos arreglado un fallo que teniamos en frontend con las imagenes.
También vamos a probar si funciona ahora bien el pillar el cliente con el token

@fravilpae fravilpae linked an issue Mar 28, 2024 that may be closed by this pull request
Copy link
Contributor

@adrrf adrrf left a comment

Choose a reason for hiding this comment

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

todo en orden

Para no complicarnos con el fallo que nos estaba dando en frontend, hemos decidido quitar los datos que se pillaban desde fuera y ahora solo te pilla el ocialClient. Hemos hecho un metodo nuevo que te devuelve los datos de DjangoUser dado el token, y de ahí pillaremos el username y el email cuando haga falta
@fravilpae fravilpae force-pushed the fix/92-errores-varios branch from aa95cf5 to 4eeae2b Compare March 29, 2024 10:33
Copy link

Quality Gate Passed Quality Gate passed

Issues
1 New issue
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
10.3% Duplication on New Code

See analysis details on SonarCloud

@adrrf adrrf merged commit c7e7508 into develop Mar 29, 2024
9 checks passed
@adrrf adrrf deleted the fix/92-errores-varios branch March 29, 2024 10:35
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

FIX: Errores varios
3 participants