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

#8667: Modified ClientPayment #1622

Merged
jsegarra merged 15 commits from 8667-FixClientNewPayment into dev 2025-03-31 08:17:36 +00:00
Conversation 0 Commits 15 Files Changed 2 +26 -27
jon commented 2025-03-24 08:08:58 +00:00
Member
Copy Link
No description provided.
jon added 1 commit 2025-03-24 08:08:58 +00:00
gitea/salix-front/pipeline/pr-dev There was a failure building this commit Details
4460b7110e refactor: refs #8667 modified client payment to be the same as Salix
jon added 1 commit 2025-03-24 08:11:01 +00:00
gitea/salix-front/pipeline/pr-dev There was a failure building this commit Details
9822ba6fc8 Merge branch 'dev' into 8667-FixClientNewPayment
jon added 1 commit 2025-03-24 09:33:24 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
ac2fbad26b Merge branch 'dev' into 8667-FixClientNewPayment
jon requested review from jsegarra 2025-03-24 13:44:01 +00:00
jsegarra requested changes 2025-03-25 08:32:46 +00:00
Dismissed
src/pages/Customer/Card/CustomerBalance.vue Outdated
@ -256,2 +251,2 @@
<template #column-balance="{ rowIndex }">
{{ toCurrency(balances[rowIndex]?.balance) }}
<template #column-balance="{ row }">
{{ toCurrency(balances[row]?.balance) }}
jsegarra commented 2025-03-25 08:31:26 +00:00
Member
Copy Link

esto no puede ir en format?
Veo que tiene dependencia con balances pero yo creo que al hacerlo en el onfetch todavía no renderiza

esto no puede ir en format? Veo que tiene dependencia con balances pero yo creo que al hacerlo en el onfetch todavía no renderiza
src/pages/Customer/Card/CustomerBalance.vue
@ -258,0 +252,4 @@
{{ toCurrency(balances[row]?.balance) }}
</template>
<template #column-workerFk="{ row }">
<span class="link">
jsegarra commented 2025-03-25 08:30:56 +00:00
Member
Copy Link

@click.stop

@click.stop
jon marked this conversation as resolved
jon added 2 commits 2025-03-25 10:24:46 +00:00
jon added 1 commit 2025-03-25 10:50:59 +00:00
gitea/salix-front/pipeline/pr-dev There was a failure building this commit Details
7329d096d2 fix: refs #8667 balance field
jon requested review from jsegarra 2025-03-25 10:51:45 +00:00
jon added 1 commit 2025-03-25 11:34:53 +00:00
gitea/salix-front/pipeline/pr-dev There was a failure building this commit Details
31c7b83ea5 Merge branch 'dev' into 8667-FixClientNewPayment
jon added 1 commit 2025-03-25 12:07:24 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
071145a998 Merge branch 'dev' into 8667-FixClientNewPayment
jsegarra requested changes 2025-03-25 22:19:32 +00:00
Dismissed
src/pages/Customer/components/CustomerNewPayment.vue
@ -48,6 +48,7 @@ const maxAmount = ref();
const accountingType = ref({});
const isCash = ref(false);
const formModelRef = ref(false);
jsegarra commented 2025-03-25 22:11:03 +00:00
Member
Copy Link

Sobra el import de FormModel

Sobra el import de FormModel
jon marked this conversation as resolved
src/pages/Customer/components/CustomerNewPayment.vue Outdated
@ -122,3 +123,3 @@
});
if (viewReceipt.value) openReport(`Receipts/${id}/receipt-pdf`);
if (viewReceipt.value) openReport(`Receipts/${id}/receipt-pdf`, {}, '_blank');
jsegarra commented 2025-03-25 22:10:27 +00:00
Member
Copy Link

Prefiero usar useOpenURL
https://redmine.verdnatura.es/issues/8810

Prefiero usar useOpenURL https://redmine.verdnatura.es/issues/8810
jon marked this conversation as resolved
src/pages/Customer/components/CustomerNewPayment.vue Outdated
@ -248,3 +248,3 @@
</div>
<VnRow>
<VnInputNumber
<VnInput
jsegarra commented 2025-03-25 21:59:32 +00:00
Member
Copy Link

VnAccountNumber?

VnAccountNumber?
jon added 3 commits 2025-03-26 11:54:32 +00:00
jon added 1 commit 2025-03-26 13:27:43 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
e44b32f789 refactor: refs #8667 corrected function name and value
jon requested review from jsegarra 2025-03-26 13:46:54 +00:00
jon added 1 commit 2025-03-27 12:37:27 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
c743ad4ac1 Merge branch 'dev' into 8667-FixClientNewPayment
jsegarra added 1 commit 2025-03-27 20:16:41 +00:00
jsegarra approved these changes 2025-03-27 20:16:47 +00:00
jsegarra scheduled this pull request to auto merge when all checks succeed 2025-03-27 20:16:56 +00:00
jon added 1 commit 2025-03-31 07:54:46 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
6cee67733d Merge branch 'dev' into 8667-FixClientNewPayment
jsegarra merged commit 8c75ec41d8 into dev 2025-03-31 08:17:35 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
jsegarra
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#1622
No description provided.
Delete Branch "8667-FixClientNewPayment"

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?