#7874 add observation type #765
No reviewers
jsegarra
Labels
No Milestone
No Assignees
3 Participants
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: verdnatura/salix-front#765
Loading…
Reference in New Issue
No description provided.
Delete Branch "7874-addObservationType"
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?
#7874 add observation typeto WIP: #7874 add observation typeWIP: #7874 add observation typeto #7874 add observation typeNo me funciona
Necesitas las ramas de back y front
Yo lo veo bien, pero me permite darle al botón de guardar si me dejo alguno de los dos campos
Desde consignatario me causa confusión ver verde y rojo. Entiendo que el verde es para la información diferente a notas y lo rojo para notas
Conociendo mas a los usuarios, a futuro nos reportaran que los mensajes causan confusión
Las notas de consignatario no son las mismas que clientObservations, eso sería otra tarea.
Lo otro ok.
@ -240,39 +240,33 @@ function handleLocation(data, location) {class="row q-gutter-md q-mb-md"v-for="(note, index) in notes">Aqui solo quito los divs y mejoró la ui, pero porque era algo sencillo, esta tarea no va realcionada con esta sección.
@ -25,3 +30,3 @@const quasar = useQuasar();const currentUser = ref(state.getUser());const newNote = ref('');const newNote = reactive({ text: '', observationTypeFk: null });porque no puede ser null?
@ -125,0 +159,4 @@option-label="description"style="flex: 0.15"densereadonlyPorque usamos vnselect para informar al usuario del tipo? Entendería esto si de alguna manera cambiase de estado, es decir readonly o no.
Mi propuesta es que sea un texto o por ejemplo un QBadge outlined
Veo consigo usar vnselect readonly
Es para seguir el patrón de estilo de otras secciones. @jgallego que opinas?
@ -32,3 +31,1 @@return { clientFk: item.clientFk, text: newObservation.value };});await axios.post('ClientObservations', data);const observationTypeFk = (Cada vez que hacemos un submit lanzamos una petición que siempre será la misma?
Fuera no lo veo porque haríamos una llamada cuando no es necesario pero dentro tampoco. Quizás algo intermedio tipo, si ya has hecho la petición una vez no la hagas otra
Al ser un dialogo , una vez se cierra ya no se puede usar(o no de la manera en que lo usamos nosotros) y se abre uno nuevo la proxima vez. De todas maneras pongo la comprobación.
Estilo actual @jgallego
Propuesta
Que lo muevas a la parte superior me gusta porque ganamos espacio, pero yo pondria el texto "tipo de observacion" o al menos "tipo" y al lado finanzas (traducido) revisa si los tipos se traducen.
No tienen traducción
Le he puesto Tipo con otro color
Opcion A:
Opcion B:
yo usaria el mismo que en item
@ -151,2 +188,4 @@background-color: $primary;}.vn-row > :nth-child(2) {El vnSelect
y yo ya lo veo bien
@ -12,36 +12,40 @@ import VnPaginate from 'components/ui/VnPaginate.vue';import VnUserLink from 'components/ui/VnUserLink.vue';import VnConfirm from 'components/ui/VnConfirm.vue';import VnAvatar from 'components/ui/VnAvatar.vue';import VnRow from './VnRow.vue';Podemos seguir el standard de las lineas superiores, queda raro 2 formatos en el mismo archivo
New commits pushed, approval review dismissed automatically according to repository settings