verdnatura/salix
verdnatura
/
salix
3
5
Fork 0
Code Issues Pull Requests 62 Releases Wiki Activity

refs #6757 update and modify triggers UPPER supplier #1960

Closed
ivanm wants to merge 3 commits from 6757-Modify-UPPER-supplier into dev
pull from: 6757-Modify-UPPER-supplier
merge into: verdnatura:dev
Conversation 1 Commits 3 Files Changed 1 +3
ivanm commented 2024-01-29 12:47:43 +00:00
Member
Copy Link
No description provided.
ivanm added 1 commit 2024-01-29 12:47:44 +00:00
gitea/salix/pipeline/head There was a failure building this commit Details
4781359d6e refs #6757 update and modify triggers UPPER supplier
ivanm requested review from guillermo 2024-01-29 12:48:21 +00:00
ivanm added 1 commit 2024-01-30 09:42:32 +00:00
gitea/salix/pipeline/head There was a failure building this commit Details
56296ad10b refs #6757 fix test
guillermo requested changes 2024-01-31 08:22:36 +00:00
db/routines/vn/triggers/supplier_beforeInsert.sql Outdated
@ -4,5 +4,7 @@ CREATE OR REPLACE DEFINER=`root`@`localhost` TRIGGER `vn`.`supplier_beforeInsert
FOR EACH ROW
BEGIN
SET NEW.editorFk = account.myUser_getId();
SET NEW.name = UPPER(NEW.name);
guillermo commented 2024-01-31 08:22:34 +00:00
Member
Copy Link

Se ha descartado poner un UPPER en los triggers

Se ha descartado poner un UPPER en los triggers
ivanm added 1 commit 2024-01-31 11:44:12 +00:00
gitea/salix/pipeline/pr-dev This commit looks good Details
3f9a6ca3fe refs #6757 return original triggers and fix test
ivanm commented 2024-02-01 07:39:28 +00:00
Author
Member
Copy Link

He dejado solo el UPDATE, así lo podemos subir y dejamos la parte de BBDD del redmine hecha. La parte de js se la asignó Javi a Carlos Satorres mientras hablábamos ayer.

He dejado solo el UPDATE, así lo podemos subir y dejamos la parte de BBDD del redmine hecha. La parte de js se la asignó Javi a Carlos Satorres mientras hablábamos ayer.
ivanm requested review from guillermo 2024-02-01 07:39:33 +00:00
guillermo refused to review 2024-02-01 07:42:57 +00:00
guillermo closed this pull request 2024-02-26 10:33:25 +00:00
All checks were successful
gitea/salix/pipeline/pr-dev This commit looks good
Required
Details

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No reviewers
guillermo
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
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#1960
No description provided.
Delete Branch "6757-Modify-UPPER-supplier"

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?