PR-3 #160
Closed
hyervoni
wants to merge 0 commits from
(deleted):PR-3
into dev
pull from: (deleted):PR-3
merge into: verdnatura:dev
verdnatura:dev
verdnatura:8277-createEntryControl
verdnatura:8443-vehicleEvent
verdnatura:fix-CustomerConsumption
verdnatura:6958-routesMonitor
verdnatura:8050-AddWorkerManagement
verdnatura:e2e_monitor
verdnatura:8422-createVehicleDms
verdnatura:8193-AddFilterInClientNotes
verdnatura:8441-createVehicleInvoiceInSection
verdnatura:8824-addGradeCredit
verdnatura:8828-filterVnSelectWarehouses
verdnatura:test
verdnatura:hotFix_vnJsonValue_showHours
verdnatura:Fix-SummaryCardsFlex
verdnatura:8363-RefactorItemFixedPrice
verdnatura:8116-ChangeTaxDataCheckedField
verdnatura:try_testIsolation_false
verdnatura:7516-refactorNavigateAndRedirect
verdnatura:master
verdnatura:8725_submit_form_onClick
verdnatura:warmfix_ticketList_sortByShipped
verdnatura:7984-addCurrency
verdnatura:8115-parkingMassiveCreate
verdnatura:8406-crudModelUpdate
verdnatura:update_vitest_version
verdnatura:8016-userRole
verdnatura:warmfix_vnLinkPhone
verdnatura:8006_eslint_rules
verdnatura:8647_fix_warnings
verdnatura:5835-hotfix-icon
verdnatura:8441-warmfix-includeTest
verdnatura:6696-manaAfterDepartment
verdnatura:7356_e2e
verdnatura:6564-enhanceTicketAdvance
verdnatura:8326-warmfix-preventNull
verdnatura:8449-fixes_2
verdnatura:6683-createPropertiesSection
verdnatura:Warmifx-SummaryCards
verdnatura:fix_itemFilter_multiLine
verdnatura:7638-showAllCmrs
verdnatura:warmFix_remove_keepData
verdnatura:fix_vnCard_url
verdnatura:7995-CreateHasAcl
verdnatura:hotfix_ticketSale_handle_notifications
verdnatura:Fix-MakeTestsFail
verdnatura:8217_mapper
verdnatura:hotfix_remove_reserved
verdnatura:8449-vnLogFilter
verdnatura:8440-createVehicleNotes
verdnatura:8667-FixClientNewPayment
verdnatura:6696-mana
verdnatura:8673-viewEntryNotesField
verdnatura:warmfix_monitorClients
verdnatura:8655-scrollUpButton
verdnatura:7701-FormModelUseUrl
verdnatura:hotfix_ticketSale_hasChanges
verdnatura:8715-jenkins_changelog
verdnatura:7551-quasarBuildWarnings
verdnatura:8772-align_linkPhone
verdnatura:hotfix_rowClickActions
verdnatura:8118-createComponentVnDropdown
verdnatura:8790-smsPredefinedText
verdnatura:minor_requested_changes_invoiceOutList
verdnatura:hotfix_ticket_basicdata
verdnatura:beta
verdnatura:8296-sistema-de-warnings
verdnatura:7830-fixCustomerList
verdnatura:4774-traducciones
verdnatura:8621-createCmrListE2eTest
verdnatura:warmFix_FixResponisveCreateForms
verdnatura:hotfix_negative_available
verdnatura:8131-filer-style
verdnatura:8691_arrayData_warning_setup
verdnatura:fix_workerDescriptorMenu_i18n
verdnatura:8709-workerCalendarFixes
verdnatura:8745-fixCustomer
verdnatura:Warmfix-DepartmentIcon
verdnatura:7869-ModifyZones
verdnatura:8388-createInvoiceInVehicle
verdnatura:Hotfix-RouteSendSms
verdnatura:7449-claimBeginningQuantity
verdnatura:8721-reorderColumnsRouteTickets
verdnatura:8581-e2eInvoiceIn
verdnatura:arrayData_warning_setup
verdnatura:6943_customerFD_alreadyExists
verdnatura:8700-InvoiceOutRef
verdnatura:warmfix_-simplify_ticketListFilter
verdnatura:hotfix_ticketNegative
verdnatura:8041-unifyClassLink
verdnatura:7587-addClaimedTicketRefToRefundTicket
verdnatura:6452-useIdExtension
verdnatura:8648-addTestCasesInRoadmapListTest
verdnatura:8664-refactorCmrList
verdnatura:8472-unifyStileForMoreCreateDialogSlot
verdnatura:8045-FixIconDescriptorRedirect
verdnatura:warmfix_ticketNegative
verdnatura:warmfix_ticketFilter
verdnatura:8698_jenkins_e2e_onlyChanges
verdnatura:8074-AddLoaderInPaginate
verdnatura:hotfix_fixGetAgenciesTicketList
verdnatura:Warmfix-ZoneDistributionPoint
verdnatura:8370-WorkerTimeControl
verdnatura:hotfix_agencyModes_sortBy
verdnatura:8349-e2eItemFixedPrice
verdnatura:warmFix-nextTick
verdnatura:warmfix_vnNotes_filter
verdnatura:fix_ticketNegatives
verdnatura:warmfix-workerBusiness
verdnatura:hotfix_agency_autoSelected
verdnatura:warmfix_descriptorProxy_dataKey
verdnatura:hotfix_customerBalance_newPayment
verdnatura:claimAgile-test
verdnatura:6553-workerBusinessV2
verdnatura:7793_sortByWeight
verdnatura:hotfix_ticketSale_quantity
verdnatura:fix_e2e_ticketSale_claim
verdnatura:7347_supplierConsumption
verdnatura:hotfix_ticketSale_other
verdnatura:6897-warmfix-destroyInput
verdnatura:6321_improve_handleResults
verdnatura:hotfix_double_ticketSale
verdnatura:8388-fixUnbook
verdnatura:warmfix_style_vnTable_card
verdnatura:6695-e2e_parallel_inLocal
verdnatura:fix_frenchMothersDay_i18n
verdnatura:6943_re_propagate
verdnatura:warmfix_vnTable_card_descriptor
verdnatura:6441-claimPack
verdnatura:6897-entryBuyListRefactor
verdnatura:7408-VnComponents
verdnatura:hotfix_customerFD_mapper
verdnatura:fix_customerConsumption_filter
verdnatura:warmfix_reload_scriptIsMissing
verdnatura:8335-creationFormErrors
verdnatura:Fix-ItemFixedPriceNameFilter
verdnatura:7731-clientViesCode
verdnatura:7248-tablesObservation
verdnatura:7058_leftMenu_vitest
verdnatura:Fix-MonitorTicketSummaryDialog
verdnatura:7405-vueDocs
verdnatura:7408-importCustomComponents
verdnatura:7351-claimSummaryLines
verdnatura:hotfix_customerExtendedList
verdnatura:6972_defaultGlobalValues
verdnatura:8339_fix_preview
verdnatura:hotfix_customerDescriptor_orderBtn
verdnatura:7308_warning
verdnatura:update_quasar_version
verdnatura:hotfix_newCustomer_SalesPerson
verdnatura:warning_defaultValue_firstParam
verdnatura:Hotfix-RedirectToSalesFromOrder
verdnatura:Fix-RedirectToTicketSale
verdnatura:7319-warningClaimLog
verdnatura:8077-sumDefaulterFront
verdnatura:6972-exteddnComponents
verdnatura:hotfix_vnLocation_selectValue
verdnatura:7069-vnAccountNumber
verdnatura:8225-UseSameMoreOptions
verdnatura:7936-hotfix-refactorCaption
verdnatura:Fix-IntegrationE2ETests
verdnatura:7202-AddCustomAgentFkColumn2
verdnatura:Fix-ItemFixedPrice
verdnatura:7202-AddCustomAgentFkColumn
verdnatura:Hotfix-ZoneEventSelect
verdnatura:Hotfix-VnSms
verdnatura:hotfix_customerConsumption_campaignFilter
verdnatura:validate_translations
verdnatura:warmfix_noUsedVars
verdnatura:Hotfix-VnSearchbarRemoveParams
verdnatura:Hotfix-VnTableBottomButton
verdnatura:hotfix_customerSampleCreate
verdnatura:7301-removeRedundantInventories
verdnatura:8245-comportamientoListadoModulos
verdnatura:VnSearchbar-SearchRemoveParams
verdnatura:fix_orderCatalog_reload
verdnatura:7366-hotfixTravel
verdnatura:hotfix-hideMenu
verdnatura:8113-preventDuplicateRecords
verdnatura:8283_riskEvaluation
verdnatura:hotfix_improve_cp
verdnatura:hotfix_minor_24-50
verdnatura:fix_js_e2e_test
verdnatura:hotfix-addDeptMail
verdnatura:hotfix_itemFixedPrice
verdnatura:dataCy_directive
verdnatura:hotfix_ticketSale_concept
verdnatura:hotfix_vnLocation_change_value
verdnatura:7356_ticket_missing_e2e
verdnatura:hotfix_itemFixedPrice_params
verdnatura:hotfix_required_sage
verdnatura:fix_city_from_province
verdnatura:6896_order_missing_e2e
verdnatura:7354_zone_missing_e2e
verdnatura:7283_item_missing_e2e
verdnatura:6943_improve_sections_and_e2e
verdnatura:hotfix-6899InvoiceOutTable
verdnatura:hotfix-6943CustomerWebAccess
verdnatura:Hotfix--translationsTravel
verdnatura:7234-driverRouteEmail
verdnatura:setWeight_modal
verdnatura:hotfix_8217_updateCustomerCredit
verdnatura:hotfix_ticketSale_enter
verdnatura:8061_improve_newCP
verdnatura:hotfix_remove_moreOptions
verdnatura:hotfix_saySimple
verdnatura:hotfix_addressSelect_optionFormat
verdnatura:8061_newCP
verdnatura:Fix-ZoneLocationsTree2
verdnatura:Fix-ZoneLocations
verdnatura:6943_customer_improvements
verdnatura:hotFix_orderCatalogFilter_refactor_fix
verdnatura:8217_updateCustomerCredit
verdnatura:warmfix_itemLastEntriesFilter
verdnatura:fix_js_e2e
verdnatura:hotFix_customer_balance_email
verdnatura:hotfix-itemLastEntries
verdnatura:8138-add-component-ticketProblems
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:7132-FixTranslations
verdnatura:fixEntrySummary
verdnatura:7119-vehicle_module
verdnatura:hotfix-termo
verdnatura:7790_formFocus_mixin
verdnatura:7950-cmr
verdnatura:7220_cypressUnitTest
verdnatura:6839-newUIMenu
verdnatura:6818-saySimple
verdnatura:6726-LogSection
verdnatura:improve_isRequired_input
verdnatura:7925-removeRedirect
verdnatura:8101-loadMoreData
verdnatura:7940-removeEvent
verdnatura:8110_itemDiary_scroll
verdnatura:8087-nuevoCampoTravel
verdnatura:warmfix_createNewPostcodeForm
verdnatura:8010-fix_performance
verdnatura:7925_fix_cast_param_id
verdnatura:7679_improve_newPostcode_form
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: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:5938_filterGrouped
verdnatura:7409-workerBalance_fixes
verdnatura:6156-advanceTicketsNewField
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:7648_myEntries_filter
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: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
Reference in New Issue
No description provided.
Delete Branch "(deleted):PR-3"
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?
@ -35,2 +35,3 @@
const onCityCreated = async () => {
const onCityCreated = async ({ name, provinceFk }, formData) => {
await townsFetchDataRef.value.fetch();
formData.townFk = townsLocationOptions.value.find((town) => town.name === name).id;
Aqui me referiero por ejemplo https://gitea.verdnatura.es/verdnatura/salix-front/pulls/160/files#issuecomment-39480
Este no seria un enfoque correcto
Y en el caso de que se quiera hacer un filtrado por el nombre,
seria mucho mas eficiente hacer la peticion al backend pasandole el filtro que recojer todos los registros de la tabla (en produccion estas tablas tienen miles de registros).
Que como se esta haciendo ahora de recojer todo y filtrar en el frontend.
De hecho en nuestro dev (creo que lo deberiais tener en vuestra rama tambien) hemos puesto para que por ejemplo a await townsFetchDataRef.value.fetch(); le puedas pasar filter.
Ver:
src/components/FetchData.vue la funcion fetch
Pero en este caso ya estamos trabajando en un enfoque mejor para el tema de codigos postales/ciudad/provincia/etc Este PR https://gitea.verdnatura.es/verdnatura/salix-front/pulls/154/files
Nos encargaremos nosotros de sustituir estos partes por el nuevo componente cuando lo tengamos
@ -136,2 +136,3 @@
response = await axios.patch($props.urlUpdate || $props.url, body);
}
emit('onDataSaved', formData.value);
emit('onDataSaved', formData.value, response);
Aqui no he visto que se use nunca el segundo parametro dado que donde se recoja el evento se deberia poner algo como:
@on-data-saved="(...args) => onDataSaved(...args)"
Pero viendo los resultados de response, igual seria mejor devolver siempre response.data
Pq ahi esta el id del nuevo registro y luego donde se usa no hace falta hacer finds por el nombre
Corregido:
7dcdaff0ef
@ -0,0 +44,4 @@
return ''
}
const date = new Date(dateString || '');
return `${date.getUTCHours().toString().padStart(2, '0')}:${date.getUTCMinutes().toString().padStart(2, '0')}`;
Javascript tiene una forma nativa de formatear horas:
Podria quedarse algo asi (tal cual este codigo tiene un fallo que suma una hora más pero se podrá arreglar seguro):
info:
https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Date/toLocaleTimeString#try_it
https://stackoverflow.com/questions/17913681/how-do-i-use-tolocaletimestring-without-displaying-seconds
Corregido:
32b3d36a6a
@ -0,0 +10,4 @@
hours = hours ? hours : 12;
let minutes = dateHour.getUTCMinutes();
minutes = minutes < 10 ? minutes.toString().padStart(2, '0') : minutes;
Creo que es parecido a https://gitea.verdnatura.es/verdnatura/salix-front/pulls/160/files#issuecomment-39484
Hay formas nativas en javascript para obtener el formato 00:00
Corregido:
32b3d36a6a
@ -1,0 +44,4 @@
],
};
const setProvince = (provinceFk) => {
Pasa lo mismo de https://gitea.verdnatura.es/verdnatura/salix-front/pulls/160/files#issuecomment-39481
Lo sustituiremos nosotros por el nuevo enfoque
@ -4,0 +166,4 @@
background-color: var(--vn-light-gray);
}
}
.consignees-card-icon {
Seria algo parecido a poner donde se usa:
class="ml-xs mr-sm flex items-center"
Ver: https://quasar.dev/layout/grid/introduction-to-flexbox#alignment
También por si os sirve este recurso es el que suelo usar para tener una idea de que quiero y como buscarlo en quasar:
https://somospnt.com/blog/148-flexbox-que-es-y-para-que-sirve
Corregido:
9c2267e116
@ -3,1 +139,4 @@
</template>
<style lang="scss" scoped>
.col-content {
class="p-[xs, sm, ...] "
Y para el border radius (no lo he probado) pero se puede usar algo de quasar:
https://quasar.dev/style/other-helper-classes#border-related
Corregido:
9d4d3bf84d
@ -1,0 +24,4 @@
const postcodesOptions = ref([]);
const onPostcodeCreated = async ({ code, provinceFk, townFk, countryFk }, formData) => {
await postcodeFetchDataRef.value.fetch();
Pasa lo mismo de https://gitea.verdnatura.es/verdnatura/salix-front/pulls/160/files#issuecomment-39481
Lo sustituiremos nosotros por el nuevo enfoque
@ -3,1 +82,4 @@
</template>
<style lang="scss">
.consignees-card {
Pasar a estilos de quasar si se puede, por arriba he dejado links de como se pueden hacer algunas cosas
Corregido:
30c90e534c
@ -40,2 +41,2 @@
const onPostcodeCreated = async () => {
postcodeFetchDataRef.value.fetch();
const onPostcodeCreated = async ({ code, provinceFk, townFk, countryFk }, formData) => {
await postcodeFetchDataRef.value.fetch();
Pasa lo mismo de https://gitea.verdnatura.es/verdnatura/salix-front/pulls/160/files#issuecomment-39481
Lo sustituiremos nosotros por el nuevo enfoque
@ -0,0 +17,4 @@
const { t } = useI18n();
const route = useRoute();
const formInitialData = reactive({
Esto es mas una duda.
Esto no seria lo mismo que poner:
Corregido:
2400eca1e3
@ -0,0 +48,4 @@
});
const onPostcodeCreated = async ({ code, provinceFk, townFk }, formData) => {
await postcodeFetchDataRef.value.fetch();
Pasa lo mismo de https://gitea.verdnatura.es/verdnatura/salix-front/pulls/160/files#issuecomment-39481
Lo sustituiremos nosotros por el nuevo enfoque
@ -0,0 +35,4 @@
});
const onPostcodeCreated = async ({ code, provinceFk, townFk }, formData) => {
await postcodeFetchDataRef.value.fetch();
Pasa lo mismo de https://gitea.verdnatura.es/verdnatura/salix-front/pulls/160/files#issuecomment-39481
Lo sustituiremos nosotros por el nuevo enfoque
@ -0,0 +20,4 @@
const currenciesOptions = ref([]);
</script>
<template>
<FetchData
Aquí pasa parecido a lo comentado arriba. Descargarse todos los registros no es viable dado que la tabla es muy grande.
Hice código que puede valer para aquí también.
Pero igual vale la pena "incrustarlo" en el componente de VnSelectFilter, así no repetimos codigo.
Pero no hagáis aun nada un pq quiero reunirme con @juan para ver como lo podemos enfocar.
Os comento mas tarde por el grupo. Pero por dejarlo apuntado
@ -0,0 +228,4 @@
componentProps: {
title: t('Confirm deletion'),
message:
rowsSelected.value.length > 1
Creo que esto podría funcionar:
Corregido, aunque tuve que usar un ternary operator para lograrlo, ya que de esa manera cuando el lenght no era mayora a 1 te concatenaba un
false
al string.Commit:
c7b9f4ce5b
@ -0,0 +204,4 @@
</template>
<style scoped>
.list {
Revisar esto, lo he visto repetido en todos los filters.
Igual sepodria definir el ancho por defencto de VnFilterPanel y asi evitar ponerlo en todos los [Section]Filter
Corregido.
Parecen demasiados cambios pero en realidad lo que se hizo fue mover el elemento
<QList dense class="list q-gutter-y-sm q-mt-sm">
dentro del componenteVnFilterPanel
y quitarlo para no en todos los Filters que había, así se evita ser repetitivos con esto.Commit:
def175bee2
@ -0,0 +498,4 @@
width: 100%;
}
.table-actions {
Pasar a estilos de quasar
Corregido:
e9e7cf5616
@ -2,0 +90,4 @@
</template>
<style lang="scss" scoped>
.card-list {
Aqui tambien se suele repitir estose podria o poner en el componente o igual crear una clase
Sugerencia tomada, se movieron las propiedades de
card-list
a una clase global y se aplico en los lugares correspondientes.Commit:
4731bdbabd
@ -0,0 +102,4 @@
@on-fetch="(data) => (vehicleList = data)"
auto-load
/>
<FormModel
El formulario tiene los campos en inglés
Corregido:
5bb50300ce
No está corregido, los campos están desordenados y aparece "Create"
Corregido: 5726ed76725eb99f070f7b874385c679c38d19f6
La etiqueta del campo de Fecha de creación sigue apareciendo como Created.
La posición es correcta pero el label no
@ -0,0 +262,4 @@
>
<template #body="{ rows }">
<div class="q-pa-md">
<QTable
Los registros parecen estar ordenados por ID desc, sin embargo, la tabla no lo refleja en la cabecera
Corregido:
bc9ac631a5
@ -0,0 +458,4 @@
</template>
<template #body-cell-actions="props">
<QTd :props="props">
<div class="table-actions">
Los tooltips están en inglés
Corregido:
9f826ebe1d
@ -0,0 +459,4 @@
<template #body-cell-actions="props">
<QTd :props="props">
<div class="table-actions">
<QIcon
No tiene el método @click
Esta funcionalidad se corregira como parte del ticket: https://mindshore-verdnatura.atlassian.net/browse/SCRUM-145
@ -1 +1,192 @@
<template>Supplier consumption</template>
<script setup>
El texto que aparece en la VnSearchbar está en inglés
Corregido:
862dc20bcd
@ -132,3 +142,3 @@
New postcode: Nuevo código postal
Please, ensure you put the correct data!: ¡Por favor, asegúrese de poner los datos correctos!
City: Ciudad
City: Población
Se ha cambiado Ciudad por población, es correcto?
Porque en el modal de creación el titulo pone "Nuevo código de ciudad"
Si, a mi también me llamó la atención, lo hice de esa manera porque en el salix lo vi así, el label del
Select
dice "Población" y el título dice "Nueva ciudad"Parece que en test y prod aparece Ciudad, mientras que en local pone Población.
No se que impacto tienen de cara a los usuarios, pero puede que sea error.
Propuesta, si no quita tiempo, estandarizar a Ciudad ya que hay 2 ocurrencias frente a Población.
Cualquier cosa lo comentamos.
Doy por cerrado el comentario, independientemente de lo que se haga.
@ -3,0 +73,4 @@
v-model="data.bankEntityFk"
>
<template #form>
<CreateBankEntityForm
Cuando creas un nuevo Swift, en el listado de opciones no aparece. Hay que refrescar la ventana para poder usar ese valor.
Corregido:
23210600ac
@ -0,0 +1,202 @@
<script setup>
Estando en basic-data de una entrada, pulsamos sobre el primer icono del descriptor(ficha del proveedor) y tiramos atrás muestra mensajes de error de Cannot read properties of null (reading 'emitsOptions')
Buenas @jsegarra, al parecer es algo que también sucede en el modulo de claims, cree la siguiente issue que dice como reproducirlo y el posible problema para atacarlo en los próximos días.
Issue: https://mindshore-verdnatura.atlassian.net/browse/SCRUM-151?atlOrigin=eyJpIjoiMTc0YjUzZTkzZjA2NDFkM2ExMTY2NzNkM2EzMjMzMzMiLCJwIjoiaiJ9
@ -0,0 +379,4 @@
</QTable>
</template>
</VnPaginate>
<QPageSticky :offset="[20, 20]">
Revisar el tooltip porque tiene un formato raro
Corregido.
Commit:
cae42486c8
@ -0,0 +197,4 @@
<VnRow class="row q-gutter-md q-mb-md">
<div class="col">
<QFile
label="Standard"
Valor literal de Standard es correcto? En Salix no aparece esto
Corregido.
Commit:
5d0290f585
@ -2,0 +128,4 @@
<QPageSticky :offset="[20, 20]">
<QBtn fab icon="add" color="primary" @click="redirectToCreateView()" />
<QTooltip>
{{ t('supplier.list.newSupplier') }}
Creo que el texto del tooltip no coincide
Corregido
Commit:
a888472ad2
@ -1 +1,192 @@
<template>Supplier consumption</template>
<script setup>
Genera un error por consola y rompe la navegación
Me di cuenta que la vista lleva un componente de filtros también.
Cree una tarea porque es un tema un poco más grande y atacarlo junto a lo que mencionas: https://mindshore-verdnatura.atlassian.net/browse/SCRUM-152?atlOrigin=eyJpIjoiMzE4ZmQzOGYwOTlkNGIyZjlmN2U0YWIzNzExYTZmYmYiLCJwIjoiaiJ9
Ya quedo todo implementado!
Commit:
f1ff128f4a
No estoy de acuerdo "ya quedo todo implementado!".
Si te fijas, en Salix, el nombre del articulo está en azul y al pulsarlo te muestra una ventana. Esto en Lilium no pasa
Si, claro mil disculpas que no lo aclaré bien antes, pero habíamos decidirlo atajarlo despúes, ya que el item descriptor es un poco más complicado que los otros descriptors.
Dejo las issues relacionadas con este comment:
Crear item descriptor
Aplicar item descriptor en lugares faltantes
@ -6,7 +6,6 @@ import { useI18n } from 'vue-i18n';
import { QCheckbox, QIcon } from 'quasar';
Termógrafo e historial no muestra "sin resultados", es correcto?
En esta PR todavía no están desarrollados los submodulos de travel, seguramente salgan para la próxima PR
@ -290,2 +259,4 @@
</QCard>
<QCard class="full-width" v-if="entriesTableRows.length > 0">
<a class="header" :href="travelUrl + 'entry'">
{{ t('travel.summary.entries') }}
Creo que algo se ha roto, porque cuando entras al detalle de un travel, y vas a la sección de Datos básicos no se muestra nada, es correcto?
De momento es correcto, todavía no desarrollamos los submodulos de Travel, entrarán para el próximo PR
@ -68,3 +69,2 @@
const onPostcodeCreated = async () => {
postcodeFetchDataRef.value.fetch();
const onPostcodeCreated = async ({ code, provinceFk, townFk }, formData) => {
El formulario aparece validado y con el campo de método de pago realizado
Corregido.
Commit:
4731bdbabd
Lo vemos, porque no veo que la solución esté bien aplicada en el commit
@jsegarra mil disculpas, pero me equivoque a la hora de dejar el commit
El commit de esta solucion es este:
7835867325
@ -357,6 +363,7 @@ onMounted(async () => {
</template>
</VnSelectCreate>
</VnRow>
<pre>{{ data }}</pre>
Etiqueta pre
Corregido
Commit:
533435c7ba
@ -1,0 +29,4 @@
},
},
],
where: { clientFk: '1' },
Está puesto el clientFk en código y por tanto carga una información diferente.
Corregido:
3fc1bff213
4731bdbabd
to23210600ac
42d41b1350
to3fc1bff213
@ -0,0 +98,4 @@
v-close-popup
/>
<QBtn
label="Ok"
Parece obvio que sea un texto tipo OK, pero mejor si seguimos el estándar
Corregido: 6060fab356dd60f2d34ccf2b68b9fadf37ebefae
@ -1,0 +21,4 @@
onMounted(() => getBankEntities());
const getBankEntities = async () => {
Esto creo que no se hace así.
He mirado en la rama dev y no me coincide con lo de esta PR
Corregido
9a23bb2ed2
6060fab356
to9a23bb2ed2
Fusionado y arreglado de conflictos en #166
Pull request closed