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

#7380 - Substitution field #2677

Merged
jsegarra merged 11 commits from 7380_substitutionCondition into dev 2024-07-09 07:46:51 +00:00
Conversation 3 Commits 11 Files Changed 2 +4 -1
jsegarra commented 2024-07-03 20:36:31 +00:00
Member
Copy Link
No description provided.
jsegarra added 2 commits 2024-07-03 21:02:54 +00:00
gitea/salix/pipeline/pr-dev There was a failure building this commit Details
17e7dc8c7e feat(salix): refs #7380 #7380 new typeObservation
jgallego requested changes 2024-07-05 11:25:28 +00:00
Dismissed
db/versions/11132-aquaDracena/00-firstScript.sql Outdated
@ -0,0 +1,4 @@
INSERT IGNORE INTO vn.observationType (`description`,code,hasNewBornMessage)
VALUES ('Sustitución','substitution',0);
ALTER TABLE vn.client ADD substitutionAllowed BOOL DEFAULT false NULL;
jgallego commented 2024-07-05 11:25:26 +00:00
Owner
Copy Link

no es necesario crear un campo, usamos las observaciones

no es necesario crear un campo, usamos las observaciones
jsegarra marked this conversation as resolved
jsegarra added 1 commit 2024-07-07 21:23:34 +00:00
gitea/salix/pipeline/pr-dev There was a failure building this commit Details
2f10066290 feat(salix): refs #7380 #7380 remove new field
jsegarra changed title from WIP: #7380 - Substitution field to #7380 - Substitution field 2024-07-07 21:23:45 +00:00
jsegarra added 1 commit 2024-07-07 21:27:08 +00:00
gitea/salix/pipeline/pr-dev There was a failure building this commit Details
570b06bd18 Merge branch 'dev' into 7380_substitutionCondition
jsegarra commented 2024-07-07 21:58:49 +00:00
Author
Member
Copy Link

@jgallego Hay un problema con este código SQL
Al ejecutarse, trunca la tabla y la tabla pasa a tener solo un registro, el del script. La configuración de la tabla tiene que el campo id es tinyint(3) y AI, sin embargo creo que esta ocurriendo por un tema de asincronia y es que primero inserta el registro con id 1 y luego las ficture.before.sql:522 tiene otro 1.

Ya me dices cuál es la mejor práctica a seguir

@jgallego Hay un problema con este código SQL Al ejecutarse, trunca la tabla y la tabla pasa a tener solo un registro, el del script. La configuración de la tabla tiene que el campo id es tinyint(3) y AI, sin embargo creo que esta ocurriendo por un tema de asincronia y es que primero inserta el registro con id 1 y luego las ficture.before.sql:522 tiene otro 1. Ya me dices cuál es la mejor práctica a seguir
jgallego commented 2024-07-08 05:38:38 +00:00
Owner
Copy Link

@jgallego Hay un problema con este código SQL
Al ejecutarse, trunca la tabla y la tabla pasa a tener solo un registro, el del script. La configuración de la tabla tiene que el campo id es tinyint(3) y AI, sin embargo creo que esta ocurriendo por un tema de asincronia y es que primero inserta el registro con id 1 y luego las ficture.before.sql:522 tiene otro 1.

Ya me dices cuál es la mejor práctica a seguir

@jsegarra añáde el registro en las fixtures, y si va todo bien cuando hagas el push haces el insert en producción

> @jgallego Hay un problema con este código SQL > Al ejecutarse, trunca la tabla y la tabla pasa a tener solo un registro, el del script. La configuración de la tabla tiene que el campo id es tinyint(3) y AI, sin embargo creo que esta ocurriendo por un tema de asincronia y es que primero inserta el registro con id 1 y luego las ficture.before.sql:522 tiene otro 1. > > Ya me dices cuál es la mejor práctica a seguir @jsegarra añáde el registro en las fixtures, y si va todo bien cuando hagas el push haces el insert en producción
👍 1
jsegarra added 4 commits 2024-07-08 07:54:59 +00:00
jsegarra added 1 commit 2024-07-08 08:03:37 +00:00
gitea/salix/pipeline/pr-dev This commit looks good Details
a9b4b3ee16 Merge branch 'dev' into 7380_substitutionCondition
jsegarra requested review from jgallego 2024-07-08 08:03:39 +00:00
jgallego requested changes 2024-07-08 10:12:39 +00:00
Dismissed
db/versions/11132-aquaDracena/00-firstScript.vn.sql
@ -0,0 +1,2 @@
INSERT IGNORE INTO vn.observationType (`description`,code)
jgallego commented 2024-07-08 08:18:11 +00:00
Owner
Copy Link

si esto fallaba antes..ahora también fallarà no? habría que quitar esta linea

si esto fallaba antes..ahora también fallarà no? habría que quitar esta linea
jsegarra commented 2024-07-08 10:13:50 +00:00
Author
Member
Copy Link

Ahora no falla porque los tests pasan

Ahora no falla porque los tests pasan
jsegarra added 1 commit 2024-07-08 12:11:56 +00:00
jsegarra requested review from jgallego 2024-07-09 06:24:19 +00:00
jsegarra added 1 commit 2024-07-09 06:24:29 +00:00
gitea/salix/pipeline/pr-dev This commit looks good Details
4d52e32718 Merge branch 'dev' into 7380_substitutionCondition
jgallego approved these changes 2024-07-09 07:42:00 +00:00
jgallego left a comment
Owner
Copy Link

ok, simplemente por asegurar lanza un myt run antes de mergear

ok, simplemente por asegurar lanza un myt run antes de mergear
jsegarra merged commit 725cbb5aa4 into dev 2024-07-09 07:46:51 +00:00
jsegarra commented 2024-07-09 07:47:00 +00:00
Author
Member
Copy Link

image

![image](/attachments/7847bc98-0f9c-403f-bd87-5ea2047edfcc)
image.png
36 KiB
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#2677
No description provided.
Delete Branch "7380_substitutionCondition"

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?