verdnatura/hedera-web
verdnatura
/
hedera-web
0
1
Fork 3
Code Issues Pull Requests 4 Releases Wiki Activity

refactor: migrate fetchUser API call from PHP to Salix #111

Merged
jsegarra merged 5 commits from ldragan/hedera-web:taro/migrate-fetchUser into beta 2025-03-13 08:08:36 +00:00
Conversation 0 Commits 5 Files Changed 1 +4 -5
1 changed files with 4 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
src/stores/user.js
View File

@ -248,11 +248,10 @@ export const useUserStore = defineStore('user', () => {
const fetchUser = async (userType = 'user') => {
try {
const userData = await jApi.getObject(
'SELECT id, nickname, name, lang FROM account.myUser'
);
if (userType === 'user') mainUser.value = userData;
else supplantedUser.value = userData;
const userData = await api.get('VnUsers/getCurrentUserData');
ldragan marked this conversation as resolved
jsegarra commented 2025-02-18 08:07:03 +00:00
Review
Copy Link

El token no es necesario porque está definido que se coja en boot/axios
De esta manera evitamos que hacer getToken y pasarlo como parametro de la petición.
Prueba a quitarlo y me dices si obtienes el mismo resultado. Si no comentamos

El token no es necesario porque está definido que se coja en boot/axios De esta manera evitamos que hacer getToken y pasarlo como parametro de la petición. Prueba a quitarlo y me dices si obtienes el mismo resultado. Si no comentamos
ldragan commented 2025-02-19 05:14:23 +00:00
Review
Copy Link

Tenés razón. Probé quitarlo y anda perfecto.

En renewToken se pasa de forma explícita, pero también se autentica con el _tokenMultimedia. Supongo que en ese caso tiene sentido, incluso si redundante, para esclarecer posibles dudas comunicando intención deliberada:

        const _token = getToken();
        const tokenData = await api.post('VnUsers/renewToken', {
            headers: { Authorization: _token }
        });

        const _tokenMultimedia = getTokenMultimedia();
        const tokenMultimedia = await api.post('VnUsers/renewToken', {
            headers: { Authorization: _tokenMultimedia }
        });
Tenés razón. Probé quitarlo y anda perfecto. En `renewToken` se pasa de forma explícita, pero también se autentica con el `_tokenMultimedia`. Supongo que en ese caso tiene sentido, incluso si redundante, para esclarecer posibles dudas comunicando intención deliberada: ```js const _token = getToken(); const tokenData = await api.post('VnUsers/renewToken', { headers: { Authorization: _token } }); const _tokenMultimedia = getTokenMultimedia(); const tokenMultimedia = await api.post('VnUsers/renewToken', { headers: { Authorization: _tokenMultimedia } }); ```
if (userType === 'user') mainUser.value = userData.data;
else supplantedUser.value = userData.data;
} catch (error) {
console.error('Error fetching user: ', error);
}