verdnatura/salix
verdnatura
/
salix
3
4
Fork 0
Code Issues Pull Requests 54 Releases Wiki Activity

feat: refs #8401 create triggers to itemTaxCountry #3370

Merged
robert merged 5 commits from 8401-itemTaxCountryTrigger into master 2025-01-21 10:37:14 +00:00
Conversation 0 Commits 5 Files Changed 2 +12
robert commented 2025-01-19 07:30:36 +00:00
Member
Copy Link
No description provided.
robert added 1 commit 2025-01-19 07:30:37 +00:00
gitea/salix/pipeline/pr-master This commit looks good Details
5fe032ac75 feat: refs #8401 create triggers to itemTaxCountry
robert requested review from carlosap 2025-01-19 07:35:44 +00:00
jgallego requested changes 2025-01-19 08:13:22 +00:00
Dismissed
db/routines/vn/triggers/itemTaxCountry_beforeDelete.sql Outdated
@ -0,0 +3,4 @@
BEFORE DELETE ON `itemTaxCountry`
FOR EACH ROW
BEGIN
SIGNAL SQLSTATE '45000'
jgallego commented 2025-01-19 08:13:20 +00:00
Owner
Copy Link

mes sencill posar ací directament call util.throw

mes sencill posar ací directament call util.throw
robert marked this conversation as resolved
robert added 1 commit 2025-01-20 10:19:30 +00:00
gitea/salix/pipeline/pr-master This commit looks good Details
cb065f42cd feat: refs #8401 change request
carlosap approved these changes 2025-01-20 10:25:04 +00:00
robert added 1 commit 2025-01-20 10:26:17 +00:00
gitea/salix/pipeline/pr-master This commit looks good Details
4b78c12ee4 Merge branch 'master' into 8401-itemTaxCountryTrigger
robert requested review from jgallego 2025-01-20 10:27:47 +00:00
jgallego requested changes 2025-01-20 11:10:38 +00:00
Dismissed
db/routines/vn/triggers/itemTaxCountry_beforeUpdate.sql Outdated
@ -5,2 +5,4 @@
BEGIN
SET NEW.editorFk = account.myUser_getId();
IF NOT(NEW.countryFk <=> OLD.countryFk) THEN
jgallego commented 2025-01-20 11:10:35 +00:00
Owner
Copy Link

el itemFk tampoc es pot canviar

el itemFk tampoc es pot canviar
robert marked this conversation as resolved
robert added 1 commit 2025-01-20 11:26:34 +00:00
gitea/salix/pipeline/pr-master This commit looks good Details
316a17304f feat: refs #8401 restriction itemFk
robert requested review from jgallego 2025-01-21 08:44:18 +00:00
jgallego approved these changes 2025-01-21 09:47:08 +00:00
robert added 1 commit 2025-01-21 10:07:30 +00:00
gitea/salix/pipeline/pr-master This commit looks good Details
77606468fd Merge branch 'master' into 8401-itemTaxCountryTrigger
robert merged commit 0e0db6591e into master 2025-01-21 10:37:14 +00:00
robert deleted branch 8401-itemTaxCountryTrigger 2025-01-21 10:37:14 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
carlosap
jgallego
Labels
Clear labels   
CR / Tests passed

Unit tests and e2e passed

  
e2e

Implement e2e

  
Missing tests

Missing tests

  
Needs refactor

   
Simple

Simple check / No CR needed

No Label
CR / Tests passed
e2e
Missing tests
Needs refactor
Simple
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#3370
No description provided.
Delete Branch "8401-itemTaxCountryTrigger"

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?