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

#8193: Added filter in ClientNotes #1609

Open
jon wants to merge 12 commits from 8193-AddFilterInClientNotes into dev
pull from: 8193-AddFilterInClientNotes
merge into: verdnatura:dev
Conversation 0 Commits 12 Files Changed 3 +175 -11
jon commented 2025-03-18 09:16:25 +00:00
Member
Copy Link
No description provided.
jon added 1 commit 2025-03-18 09:16:25 +00:00
gitea/salix-front/pipeline/pr-dev This commit is unstable Details
76b793b559 feat: refs #8193 added filter in client notes
jon added 1 commit 2025-03-18 09:40:48 +00:00
gitea/salix-front/pipeline/pr-dev This commit is unstable Details
9bf1f74061 feat: refs #8193 added notes e2e
jon added 1 commit 2025-03-18 12:48:52 +00:00
gitea/salix-front/pipeline/pr-dev There was a failure building this commit Details
c1964dc16e perf: refs #8193 e2e
jon added 1 commit 2025-03-18 12:50:05 +00:00
gitea/salix-front/pipeline/pr-dev There was a failure building this commit Details
6840c8f053 Merge branch 'dev' into 8193-AddFilterInClientNotes
jon added the
CR / Tests passed
 label 2025-03-18 13:42:08 +00:00
jon added 1 commit 2025-03-20 06:58:00 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
921af7abda Merge branch 'dev' into 8193-AddFilterInClientNotes
jon changed title from WIP: #8193: Added filter in ClientNotes to #8193: Added filter in ClientNotes 2025-03-20 07:10:11 +00:00
jon requested review from alexm 2025-03-20 07:10:25 +00:00
alexm requested changes 2025-03-21 13:43:17 +00:00
alexm left a comment
Member
Copy Link

Lo miramos y la enfocamos

Lo miramos y la enfocamos
src/components/ui/VnNotes.vue Outdated
@ -128,2 +193,4 @@
auto-load
/>
<FetchData
url="ClientObservations"
alexm commented 2025-03-21 13:42:56 +00:00
Member
Copy Link

Esto no deberia estar en un componente generico, VnNotes se usa en claim, worker, etc

Esto no deberia estar en un componente generico, VnNotes se usa en claim, worker, etc
jon marked this conversation as resolved
jon added 2 commits 2025-03-25 07:28:22 +00:00
jon added 1 commit 2025-04-01 12:23:34 +00:00
jon added 1 commit 2025-04-04 10:01:10 +00:00
gitea/salix-front/pipeline/pr-dev This commit is unstable Details
0d0433bec4 feat: refs #8193 make VnNotes use VnNotesFilter
jon added 1 commit 2025-04-04 10:06:00 +00:00
gitea/salix-front/pipeline/pr-dev Something is wrong with the build of this commit Details
94a66098c6 feat: refs #8193 added user filter e2e in VnNotes
jon requested review from alexm 2025-04-04 10:18:42 +00:00
jon added 1 commit 2025-04-07 07:09:52 +00:00
gitea/salix-front/pipeline/pr-dev Something is wrong with the build of this commit Details
ce722c56ca Merge branch 'dev' into 8193-AddFilterInClientNotes
alexm requested changes 2025-04-07 10:07:04 +00:00
alexm left a comment
Member
Copy Link

Lo estoy probadno en la claim 1
Y el primer filtro no filtra y hace 3 peticiones
image

Revisar

Lo estoy probadno en la claim 1 Y el primer filtro no filtra y hace 3 peticiones ![image](/attachments/503c7dd9-4478-4434-9169-5f813e40d763) Revisar
image.png
168 KiB
src/components/ui/VnNotesFilter.vue
@ -0,0 +42,4 @@
</script>
<template>
<FetchData
alexm commented 2025-04-07 10:03:28 +00:00
Member
Copy Link

Esto no deberia estar en un componente que se usa en mas secciones aparte de clientes.
Para lo que tratas de hacer puedes usar obtener de alguna manera los datos del arrayData y sacarte los users/workers

Esto no deberia estar en un componente que se usa en mas secciones aparte de clientes. Para lo que tratas de hacer puedes usar obtener de alguna manera los datos del arrayData y sacarte los users/workers
alexm added 1 commit 2025-04-07 12:55:07 +00:00
gitea/salix-front/pipeline/pr-dev There was a failure building this commit Details
cc05f25c3b Merge branch 'dev' into 8193-AddFilterInClientNotes
Some checks failed
gitea/salix-front/pipeline/pr-dev There was a failure building this commit
Required
Details
This pull request doesn't have enough approvals yet. 0 of 1 approvals granted.
This branch is out-of-date with the base branch
You are not authorized to merge this pull request.
Sign in to join this conversation.
Reviewers
No reviewers
alexm
Labels
Clear labels   
bug

Something is not working

  
CR / Tests passed

CR / Tests passed

  
duplicate

This issue or pull request already exists

  
enhancement

New feature

  
invalid

Something is wrong

  
Missing Tests

Missing Tests

  
wontfix

This won't be fixed

No Label
bug
CR / Tests passed
duplicate
enhancement
invalid
Missing Tests
wontfix
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
2 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#1609
No description provided.
Delete Branch "8193-AddFilterInClientNotes"

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?