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

3653-refactor(clientInsurance): rename column add Fk #997

Merged
carlosjr merged 4 commits from 3653-clientInsurance into dev 2022-06-10 07:13:39 +00:00
Conversation 0 Commits 4 Files Changed 8 +25 -6
alexm commented 2022-06-09 05:33:46 +00:00
Member
Copy Link
No description provided.
alexm added the
CR / Tests passed
 label 2022-06-09 05:33:46 +00:00
alexm added 2 commits 2022-06-09 05:33:48 +00:00
gitea/salix/pipeline/head This commit looks good Details
81f42985e4 sql
carlosjr requested changes 2022-06-09 09:01:03 +00:00
db/changes/10480-june/00-creditInsurance.sql Outdated
@ -0,0 +5,4 @@
ALTER TABLE `vn`.`creditInsurance`
ADD CONSTRAINT `creditInsurance_creditClassificationFk` FOREIGN KEY (`creditClassificationFk`)
REFERENCES `creditClassification` (`id`) ON DELETE CASCADE ON UPDATE CASCADE;
carlosjr commented 2022-06-09 08:56:34 +00:00
Contributor
Copy Link

Te falta referenciar el esquema de la tabla creditClassification

Te falta referenciar el esquema de la tabla creditClassification
alexm marked this conversation as resolved
carlosjr requested review from jgallego 2022-06-09 09:01:31 +00:00
alexm added 1 commit 2022-06-09 12:52:16 +00:00
gitea/salix/pipeline/head This commit looks good Details
417c798afd add schema
alexm requested review from carlosjr 2022-06-09 12:52:28 +00:00
jgallego approved these changes 2022-06-10 05:16:09 +00:00
alexm added 1 commit 2022-06-10 05:33:29 +00:00
gitea/salix/pipeline/head This commit looks good Details
de0f7c98fa Merge branch 'dev' into 3653-clientInsurance
carlosjr approved these changes 2022-06-10 07:13:29 +00:00
carlosjr left a comment
Contributor
Copy Link

LGTM

LGTM
carlosjr merged commit 9dbd1e6f72 into dev 2022-06-10 07:13:39 +00:00
carlosjr deleted branch 3653-clientInsurance 2022-06-10 07:13:39 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
jgallego
carlosjr
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#997
No description provided.
Delete Branch "3653-clientInsurance"

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?