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

fix: refs #6731 fix ticketRequest #2364

Merged
jon merged 1 commits from 6731-fixTicketRequestLog into dev 2024-04-30 10:19:31 +00:00
Conversation 0 Commits 1 Files Changed 8 +42 -39
jon commented 2024-04-24 08:32:08 +00:00
Member
Copy Link
No description provided.
jon added 1 commit 2024-04-24 08:32:08 +00:00
gitea/salix/pipeline/pr-dev This commit looks good Details
6706974784 fix: refs #6731 fix ticketRequest
jon requested review from jgallego 2024-04-24 08:34:14 +00:00
jgallego requested changes 2024-04-24 13:24:27 +00:00
Dismissed
db/routines/vn/procedures/ticketRequest_Add.sql
@ -1,19 +0,0 @@
DELIMITER $$
CREATE OR REPLACE DEFINER=`root`@`localhost` PROCEDURE `vn`.`ticketRequest_Add`(vDescription VARCHAR(255), vQuantity INT, vPrice DOUBLE, vTicketFk INT, vBuyerCode VARCHAR(3))
jgallego commented 2024-04-24 13:24:24 +00:00
Owner
Copy Link

me extraña ver que se elimina un proc pero no veo la alternativa que se usará a partir de ahora

me extraña ver que se elimina un proc pero no veo la alternativa que se usará a partir de ahora
jon commented 2024-04-25 05:16:30 +00:00
Author
Member
Copy Link

Estuve buscando quien usaba ese procedimiento, y en salix y lilium no se usa. Busqué también en access y tampoco se usa ahí el procedimiento, por eso lo eliminé

Estuve buscando quien usaba ese procedimiento, y en salix y lilium no se usa. Busqué también en access y tampoco se usa ahí el procedimiento, por eso lo eliminé
jon commented 2024-04-30 09:15:37 +00:00
Author
Member
Copy Link

He seguido las instrucciones de https://wiki.verdnatura.es/index.php/Deprecar_objetos_de_las_BBDD para el procedimiento que eliminé, y he visto que tampoco se usa en hedera ni en vn.warehouse

He seguido las instrucciones de https://wiki.verdnatura.es/index.php/Deprecar_objetos_de_las_BBDD para el procedimiento que eliminé, y he visto que tampoco se usa en hedera ni en vn.warehouse
jgallego marked this conversation as resolved
jgallego approved these changes 2024-04-30 10:12:53 +00:00
jon merged commit 2b9bfb5d85 into dev 2024-04-30 10:19:31 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
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
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#2364
No description provided.
Delete Branch "6731-fixTicketRequestLog"

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?