verdnatura/salix-front
verdnatura
/
salix-front
2
3
Fork 1
Code Issues Pull Requests 90 Releases Wiki Activity

feat(navBar): refs #7632 isLoading #844

Merged
alexm merged 5 commits from 7632-axios_isLoading into dev 2024-10-21 10:48:58 +00:00
Conversation 0 Commits 5 Files Changed 6 +8 -7
4 changed files with 8 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit d221b8e3c4 - Show all commits

3
src/boot/axios.js
View File

@ -29,8 +29,7 @@ const onResponse = (response) => {
const config = response.config;
stateQuery.remove(config);
const isSaveRequest = config.method === 'patch';
if (isSaveRequest) {
if (config.method === 'patch') {
alexm marked this conversation as resolved Outdated
jsegarra commented 2024-10-21 09:47:31 +00:00
Outdated
Review
Copy Link

Crear la variable y solo usarla en el if es lo mismo que ponerla dentro del if no?
El método post también hace save no?

Crear la variable y *solo* usarla en el if es lo mismo que ponerla dentro del if no? El método post también hace save no?
alexm commented 2024-10-21 09:56:16 +00:00
Outdated
Review
Copy Link

La variable ya estaba de antes, lo cambio.
No se la logica de pq alguien puso solo patch

La variable ya estaba de antes, lo cambio. No se la logica de pq alguien puso solo patch
notify('globals.dataSaved', 'positive');
}

3
src/components/NavBar.vue
View File

@ -114,9 +114,6 @@ const pinnedModulesRef = ref();
.q-header {
background-color: var(--vn-section-color);
}
.no-visible {
visibility: hidden;
}
</style>
alexm marked this conversation as resolved Outdated
jsegarra commented 2024-10-21 09:49:08 +00:00
Outdated
Review
Copy Link

Si solo tiene una propiedad, es lo mismo que hacer style, no?
También podemos definir la clase global y usarla.

Como gustes

Si solo tiene una propiedad, es lo mismo que hacer style, no? También podemos definir la clase global y usarla. Como gustes
<i18n>
en:

4
src/css/app.scss
View File

@ -288,3 +288,7 @@ input::-webkit-inner-spin-button {
color: $info;
}
}
.no-visible {
visibility: hidden;
}

5
test/vitest/__tests__/stores/useStateQueryStore.spec.js
View File

@ -11,8 +11,6 @@ describe('useStateQueryStore', () => {
const stateQueryStore = useStateQueryStore();
const { add, isLoading, remove, reset } = useStateQueryStore();
const firstQuery = { url: 'myQuery' };
alexm marked this conversation as resolved
jsegarra commented 2024-10-21 09:51:31 +00:00
Review
Copy Link

comentario: definimos baseQuery y luego ....baseQuery en cada variable?
Esto lo hemos hecho al definir columnas en VnTable, no?

comentario: definimos baseQuery y luego ....baseQuery en cada variable? Esto lo hemos hecho al definir columnas en VnTable, no?
const secondQuery = { url: 'myQuery' };
const thirdQuery = { url: 'myQuery' };
function getQueries() {
return stateQueryStore.queries;
@ -43,6 +41,9 @@ describe('useStateQueryStore', () => {
});
it('should add and remove query', async () => {
const secondQuery = { ...firstQuery };
const thirdQuery = { ...firstQuery };
add(firstQuery);
add(secondQuery);