HOTFIX: #6943 CustomerList form salesPersons options #790
Open
jsegarra
wants to merge 76 commits from
hotfix_newCustomer_SalesPerson
into master
pull from: hotfix_newCustomer_SalesPerson
merge into: verdnatura:master
verdnatura:dev
verdnatura:7936-improveInvoiceIn
verdnatura:master
verdnatura:hotfix_addressSelect_optionFormat
verdnatura:6629-addressObservation
verdnatura:test
verdnatura:4774-traducciones
verdnatura:hotfix_checbox_css
verdnatura:8219-InvoiceOutE2E
verdnatura:8113-preventDuplicateRecords
verdnatura:8004-liliumStyleRemake
verdnatura:7917-freelancerRoute
verdnatura:hotfix_8217_updateCustomerCredit
verdnatura:6943_improve_sections_and_e2e
verdnatura:8114-removeAutoAddress
verdnatura:8061_newCP
verdnatura:Fix-ZoneLocationsTree2
verdnatura:Fix-ZoneLocations
verdnatura:fix_moreOptions
verdnatura:hotfix_itemFixedPrice
verdnatura:6943_customer_improvements
verdnatura:hotFix_orderCatalogFilter_refactor_fix
verdnatura:8001-expeditionGrafana
verdnatura:8217_updateCustomerCredit
verdnatura:7134-supplierBalance
verdnatura:7301-itemLastEntries
verdnatura:warmfix_itemLastEntriesFilter
verdnatura:fix_js_e2e
verdnatura:performance_mixins
verdnatura:hotFix_customer_balance_email
verdnatura:8077-sumDefaulterFront
verdnatura:8194-VnSelectWorker
verdnatura:hotfix-itemLastEntries
verdnatura:8138-add-component-ticketProblems
verdnatura:Fix-AddSearchbarToWagonModule
verdnatura:6919-syncData
verdnatura:6389-changesMonitor
verdnatura:7304-warningCustomerBasicData
verdnatura:7301-itemHistory
verdnatura:hotFix_customerSumamry_balanceDue
verdnatura:8185-DuplicateLeftMenu
verdnatura:hotfix_OrderNew_redirectTo_catalog
verdnatura:hotfix_customer_salesPerson_Filter
verdnatura:hotfix_termograph_filter
verdnatura:7940-quasarBus_try
verdnatura:xxxx-addSupportService
verdnatura:7132-FixTranslations
verdnatura:fixEntrySummary
verdnatura:7119-vehicle_module
verdnatura:improve_vnSelectOption
verdnatura:vnselectoption
verdnatura:hotfix-termo
verdnatura:7790_formFocus_mixin
verdnatura:7950-cmr
verdnatura:6583-addDestinationFilter
verdnatura:7220_cypressUnitTest
verdnatura:6839-newUIMenu
verdnatura:6818-saySimple
verdnatura:6726-LogSection
verdnatura:improve_isRequired_input
verdnatura:7925-removeRedirect
verdnatura:6994-vnLog_descriptors
verdnatura:8101-loadMoreData
verdnatura:7940-removeEvent
verdnatura:6897-entryBuyListRefactor
verdnatura:8110_itemDiary_scroll
verdnatura:8087-nuevoCampoTravel
verdnatura:6695-docker_push
verdnatura:warmfix_createNewPostcodeForm
verdnatura:6242-AddHasRoundingField
verdnatura:8010-fix_performance
verdnatura:7925_fix_cast_param_id
verdnatura:7679_improve_newPostcode_form
verdnatura:7731-clientViesCode
verdnatura:7793_sortByWeight
verdnatura:7202-AddCustomAgentFkColumn
verdnatura:6452-dropWorkerPbx
verdnatura:7404_vnTable_style
verdnatura:7404-fixSomeStyleIssues
verdnatura:hotfix_vnImg_zoom_resolution
verdnatura:Hotfix-ZoneOrderFilters
verdnatura:fix_height_zoneLocationTree
verdnatura:FixSales
verdnatura:8062-testToMaster
verdnatura:8031_notify_dataSaved
verdnatura:masterToTest_conflicts
verdnatura:fix_8018
verdnatura:8018_location_label
verdnatura:fix_vnTable_onDataSaved
verdnatura:7679_improve_filters
verdnatura:warmfix-addSearchUrl
verdnatura:8048-subtitleToTownProvinceCity
verdnatura:6943-warmfix-preventNull
verdnatura:7553_FixTicketExpedition
verdnatura:vnInput_autoFocus
verdnatura:6321_negative_tickets
verdnatura:7671_dense_itemFixedPrices
verdnatura:7632-useStateQuery
verdnatura:7822-checkUrl
verdnatura:fix-ScopeDays2
verdnatura:fix_VnTable_infiniteScroll
verdnatura:highlight_today_qDate
verdnatura:add_shortcut_add_subSections
verdnatura:fix_supplierFD_location
verdnatura:7677_vnLocation_perf
verdnatura:7017_formModel_validations
verdnatura:6553-workerBusiness
verdnatura:5938_filterGrouped
verdnatura:7409-workerBalance_fixes
verdnatura:6156-advanceTicketsNewField
verdnatura:5926_pda_worker_docuware
verdnatura:5938_fine_ticketAdvance
verdnatura:6897-modifyEntryBuys
verdnatura:plus_key_shortcut
verdnatura:6900_simplify_invoiceInCorrective
verdnatura:7897_masterTest_24_36
verdnatura:7889_main_shortcuts
verdnatura:6772_reload_sections
verdnatura:7896_down_devToTest_2436
verdnatura:7896_devToTest_2436
verdnatura:7287-reasonLog
verdnatura:test_dev_resolveConflicts
verdnatura:7884_daysBackward
verdnatura:hotfix_account_descriptor
verdnatura:7323_workerPhoto
verdnatura:6896_order_fix
verdnatura:6336_claim_fix_states
verdnatura:7864_testToMaster_2434
verdnatura:7717_fixOrderCatalog
verdnatura:7830-customerDesplegables
verdnatura:hotfix-6449-summaryId
verdnatura:hotfix_myEntries
verdnatura:7648_myEntries
verdnatura:7323_WorkerMigration_End
verdnatura:6822-createEntryTransferOption
verdnatura:7648_myEntries_filter
verdnatura:warmfix_ticketList
verdnatura:6896_Order_End
verdnatura:6899_InvoiceOut_End
verdnatura:6825_vnTable_searchBtn_sticky
verdnatura:7380_substitutionCondition
verdnatura:7030-FixTranslations
verdnatura:6899-EndInvoiceOutMigration
verdnatura:7407-workerMedical
verdnatura:7648_customerEntries
verdnatura:7649-fixStyles
verdnatura:5447-travelExtraCommunity
verdnatura:6464-fixNegativeBases
verdnatura:6739-invoiceOut
verdnatura:6977-ClonedURL
verdnatura:7566-FixCustomerSummary
verdnatura:7604_refreshImgSumamry
verdnatura:fix-customerExtendedListFilter
verdnatura:7502-liliumIcons
verdnatura:7516_improve_navigate_redirect
verdnatura:7187-fixTranslates
verdnatura:6972_defaultGlobalValues
verdnatura:7311_boilerplate_users
verdnatura:6427_sms_resetPassword
verdnatura:6332-calendarColors
verdnatura:6767-deleteDivCol
verdnatura:ES_smsTranslations
verdnatura:7261_boilerplate_monitors
verdnatura:7239_FR_smsTranslations
verdnatura:7187-addPdaFreelancer
verdnatura:6425-translationProposal
verdnatura:6641
verdnatura:6394_filterPanel_bugs
verdnatura:6918_routerfilter_improve
verdnatura:6416-refactor-InvoiceIn
verdnatura:4466-InvoiceInCorrective
verdnatura:5888_fix_placeholder
verdnatura:6386_imageWithNoFormat
verdnatura:hotfix-amountClaimLines
verdnatura:6654_sticky_custom_toolbar
verdnatura:fix-claims
verdnatura:removeId
verdnatura:6172_transfer_ticket_message_error
verdnatura:5739-local-docker
verdnatura:6611_popup-proxy_descriptor
verdnatura:5895-vuepress
verdnatura:5895-jsdoc
verdnatura:6325-refundComercial
verdnatura:6157-createClaimAction
verdnatura:5989-redirectButton
No reviewers
Labels
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
3 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.
No due date set.
Dependencies
No dependencies set.
Reference: verdnatura/salix-front#790
Reference in New Issue
No description provided.
Delete Branch "hotfix_newCustomer_SalesPerson"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
He bajado 2 acciones del menú a botón a petición de Jesus. Estas acciones no muestran el modal dentro de cliente sino que abre una pestaña nueva en el listado de Tickets filtrados del mismo y el campo cliente preseleccionado. De igual manera para el boton de crear nuevo pedido/cesta
@ -56,3 +56,3 @@
<template>
<QItem v-ripple clickable @click="openTicketCreateForm()">
<!-- <QItem v-ripple clickable @click="openTicketCreateForm()">
los comentarios no suben
@ -98,2 +98,3 @@
icon: 'vn:lines',
action: ({ id }) => router.push({ params: { id }, name: 'TicketSale' }),
action: ({ id }) =>
window.open(
El otro día comenté contigo con Alex que el Windows open mejor no usarlo. aplica aqui?
El viernes pasado comentamos que habia un composable openReport que hacia esto.
Para este caso concreto lo ideal seria usar la etiqueta a de HTML y la propiedad hRef, pero en este caso no aplica
New commits pushed, approval review dismissed automatically according to repository settings
@ -118,6 +118,7 @@ function existSummary(routes) {
.cardSummary {
width: 100%;
max-height: 70vh;
Se quiere asi?
Lo solicita Jesus Brocal ya que se queda un espacio muy pequeño para cerrar el modal. Es incomodo.
El cirterio para darle este valor es que el modal como limite, debe respetar el alto de la barra superior
Pero cardSummary se usa en los summary no deberia tener el alto limitado
@ -21,6 +22,7 @@ const tableRef = ref();
const agencyList = ref([]);
const addressesList = ref([]);
const clientId = ref();
Si ya no se usa, quitar
@ -137,3 +139,3 @@
title: t('ticketList.summary'),
icon: 'preview',
action: (row) => viewSummary(row.id, TicketSummary),
action: (row, evt) => {
Sin hacer este codigo ya funciona asi no? (lo he probado en master)
Antes tambien funcionaba, pero lo que se quiere es que el summary se pueda abrir en una pestaña nueva, con control, auqnue creo que seria mejor hacerlo con el click de la rueda. Lo reviso
@ -0,0 +27,4 @@
Location: { val: '46000, Valencia(Province one), España', type: 'select' },
};
cy.fillInForm(data);
// cy.get(dialogInputs).eq(0).type('NAME 1');
Quitar
@ -140,0 +144,4 @@
params: { id: row.id },
name: 'TicketCard',
}).href;
window.open(url, '_blank');
Lo que queria decir es que esta funcionalidad ya esta hecha, funciona asi sin añadir ningun codigo
Si vas a https://lilium.verdnatura.es/#/customer/34655/summary y le das a
Yo lo he probado y no va en lilium.verdnatura.es Si quieres, envíame un video por privado
No me deja subir video, pero en master funciona ya el ctrl + click para abrir en una nueva pestaña
@ -28,3 +28,3 @@
'postalCode',
],
order: ['isDefaultAddress DESC', 'isActive DESC', 'nickname ASC'],
order: ['isDefaultAddress DESC', 'isActive DESC', 'id DESC', 'nickname ASC'],
Mejoramos la lista de resultados, haciendo que este ordenado por id respetando la opcion por defecto
@ -249,3 +249,3 @@
function updateStateParams() {
const newUrl = { path: route.path, query: { ...(route.query ?? {}) } };
newUrl.query[store.searchUrl] = JSON.stringify(store.currentFilter);
if (!route.path.endsWith('/summary'))
Así evitamos ahora y a futuro que si usamos VnTable en *summary, tengamos los filtros de la tabla
Añadirle un searchUrl diferente a la tabla que se use, como ya hacemos en las otras secciones
Mi no entender.
No se elimina la linea
Ya pero que la solucion seria mas en la tabla que haya en ese summary ponerle searchUrl="sumarry"
Mas que poner aqui una condicion especifica no?
Los usuarios nos dicen que para compartir el summary de un cliente ahora les sale esto en el portapapeles http://localhost:9000/#/customer/1109/summary?tickets={%22filter%22:%22{%5C%22limit%5C%22:30,%5C%22include%5C%22:[{%5C%22relation%5C%22:%5C%22ticketState%5C%22,%5C%22scope%5C%22:{%5C%22fields%5C%22:[%5C%22stateFk%5C%22,%5C%22code%5C%22,%5C%22alertLevel%5C%22],%5C%22include%5C%22:{%5C%22relation%5C%22:%5C%22state%5C%22}}},{%5C%22relation%5C%22:%5C%22invoiceOut%5C%22,%5C%22scope%5C%22:{%5C%22fields%5C%22:[%5C%22id%5C%22]}},{%5C%22relation%5C%22:%5C%22agencyMode%5C%22,%5C%22scope%5C%22:{%5C%22fields%5C%22:[%5C%22name%5C%22]}}],%5C%22where%5C%22:{%5C%22clientFk%5C%22:%5C%221109%5C%22},%5C%22order%5C%22:%5C%22shipped+DESC,+id%5C%22,%5C%22skip%5C%22:0}%22}
Lo que estan haciendo a mano es seleccionar este texto http://localhost:9000/#/customer/1109/summary
Por tanto, si me dices que hay alguna manera de montar el summary si que se añada la parte de la query, por favor, indicame como. Porque hasta ahora, he probado quitando o poniendo un valor a search-url y siempre me pone table o el valor, respectivamente.
Caso de existo, entras en un cliente y en la URL tienes esto http://localhost:9000/#/customer/1109/summary
@ -5,3 +10,3 @@
<template>
<VnInput v-bind="$attrs" v-model.number="model" type="number" />
<VnInput v-bind="$attrs" v-model.number="model" type="number" :step="step" />
Daba error en los inputs de tipo numero al introducir 2 decimales
@ -34,2 +36,2 @@
const requiredFieldRule = (val) => validations().required($attrs.required, val);
const requiredFieldRule = (val) => validations().required(isRequired.value, val);
const isRequired = computed(() => Object.keys($attrs).includes('required'));
Evitamos tener que hacer :required="true", ahora solo hace falta required
HOTFIX: CustomerList form salesPersons optionsto WIP: HOTFIX: CustomerList form salesPersons optionsWIP: HOTFIX: CustomerList form salesPersons optionsto HOTFIX: CustomerList form salesPersons options@ -538,3 +550,3 @@
(btn.show && btn.show(row)) ?? true ? 'visible' : 'hidden'
}`"
@click="btn.action(row)"
@click="handleClick($event, btn, row)"
Yo haria que te pase los datos a la función y en la función ya tenga la logica.
Estas proponiendo que la lógica de la función handleClick esté en la definición de las acciones en CustomerSummaryTable?
De todas maneras he deshecho el cambio porque al parece faltó añadir row-click
@ -220,3 +220,3 @@
@click="search()"
></QBtn>
<QForm @submit="search" id="filterPanelForm">
<QForm @submit="search" id="filterPanelForm" @keyup.enter="search()">
Pero esto ya lo tiene QForm no?
?Pero no está funcionando en master.
Al parecer en test si que va.
Los usuarios nos lo piden mucho, podemos eliminar la linea y...sabemos que se ha cambiado para que funcione correctamente?
Yo lo quitaria, puede que de problemas.
Prueba un filtro lateral que genere VnTable
Se mantiene, todavia no está el evento que lo dispara
@ -70,3 +70,3 @@
default: false,
},
appendParams: {
Usar la propiedad searchUrl a false para esto
searchUrl es una propiedad de tipo String que tiene por defecto table.
Aunque no la declares, te pone table
Si la declaras no puedes decirle que valga false, null o undefined
Ni poniendo q el type sea [string, boolean]??
Si esta solución la había planteado y probado, pero no me gustaba porque mezcla 2 tipos. Aunque veo que hay 2 resultados mas para esta combinación
Uf es q lo veo mas facil de usar que añadir un parametro mas.
Creo que poniendo el if y pasando comillas vacias haria la funcion de booleano jajja
@ -249,3 +249,3 @@
function updateStateParams() {
const newUrl = { path: route.path, query: { ...(route.query ?? {}) } };
newUrl.query[store.searchUrl] = JSON.stringify(store.currentFilter);
if (store.appendParams)
Y aqui sea if(store.searchUrl)
lo coge como String no como boolean
@ -64,1 +64,3 @@
return !isSummary.value && route.meta.moduleName && exist;
const isDialog = document.querySelectorAll('[role="dialog"]').length > 0;
// const isSameModule = `${route.meta.moduleName}Summary` === props.dataKey;
Quitar comment y console.log
@ -143,3 +171,1 @@
amount: 0,
code: companies.value.find((c) => c.id === companyId.value)?.code,
};
async function getClientRisk() {
Revisar bien la funcionalidad de este archivo, se usa en otro sitio
he creado un composable
@ -55,4 +43,3 @@
</script>
<template>
<QItem v-ripple clickable @click="openTicketCreateForm()">
Se quitan?
Se mueve la funcionalidad del DescriptorMenu a botón en Descriptor actions
@ -25,6 +25,11 @@ const $props = defineProps({
const entityId = computed(() => $props.id || route.params.id);
const customer = computed(() => summary.value.entity);
const summary = ref();
const clientUrl = ref();
No se usa no?
En quasar build no sale el warning.
Se usa en el onMounted
New commits pushed, approval review dismissed automatically according to repository settings
Arreglar conflictos
New commits pushed, approval review dismissed automatically according to repository settings
Sigue habiendo uno jeje
HOTFIX: CustomerList form salesPersons optionsto WIP: HOTFIX: CustomerList form salesPersons optionsLos test e2e no pasan
WIP: HOTFIX: CustomerList form salesPersons optionsto HOTFIX: CustomerList form salesPersons optionsHOTFIX: CustomerList form salesPersons optionsto HOTFIX: #6943 CustomerList form salesPersons optionsNew commits pushed, approval review dismissed automatically according to repository settings
@ -168,6 +172,27 @@ const setData = (entity) => (data.value = useCardDescription(entity?.name, entit
>
<QTooltip>{{ t('Customer ticket list') }}</QTooltip>
</QBtn>
<!-- <QBtn
Comentado?
@ -179,6 +204,26 @@ const setData = (entity) => (data.value = useCardDescription(entity?.name, entit
>
<QTooltip>{{ t('Customer invoice out list') }}</QTooltip>
</QBtn>
<!-- <QBtn
Comentado?
@ -28,0 +28,4 @@
const clientUrl = ref();
onMounted(async () => {
clientUrl.value = (await getUrl('client/')) + entityId.value + '/';
no se usa?
@ -180,3 +187,3 @@
search-url="tickets"
:without-header="true"
auto-load
:row-click="rowClick"
Creo que poniendo redirect ya lo hace no?
espectacular
@ -253,3 +254,4 @@
cy.get(selector).should('have.text', expectedValue);
});
Cypress.Commands.add('openActionDescriptor', (opt) => {
No se usa?
Este no
@ -256,1 +261,4 @@
1;
});
Cypress.Commands.add('openActionsDescriptor', () => {
No se usa?
Se usa en 3 archivos, ticketDescriptor e invoiceInDescriptor
@ -257,3 +265,4 @@
cy.get('.header > :nth-child(3) > .q-btn__content > .q-icon').click();
});
Cypress.Commands.add('clickButtonsDescriptor', (id) => {
Revisar que se usen los metodos
Se usa en clientList