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

4797-lilium-worker-notifications #1229

Merged
pau merged 37 commits from 4797-lilium-worker-notifications into dev 2023-02-02 08:53:29 +00:00
Conversation 0 Commits 37 Files Changed 8 +3 -1
2 changed files with 3 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 39d4b53872 - Show all commits

3
db/changes/230401/00-alter_notSubs.sql → db/changes/230401/01-alter_notSubs.sql
View File

@ -2,3 +2,6 @@ ALTER TABLE `util`.`notificationSubscription`
ADD `id` int(11) auto_increment NULL, ADD `id` int(11) auto_increment NULL,
DROP PRIMARY KEY, DROP PRIMARY KEY,
ADD CONSTRAINT PRIMARY KEY (`id`); ADD CONSTRAINT PRIMARY KEY (`id`);
ALTER TABLE `util`.`notificationSubscription`
ADD KEY `notificationSubscription_ibfk_1` (`notificationFk`);

1
db/dump/structure.sql
View File

@ -19729,7 +19729,6 @@ CREATE TABLE `notificationSubscription` (
`notificationFk` int(11) NOT NULL, `notificationFk` int(11) NOT NULL,
pau marked this conversation as resolved Outdated
alexm commented 2023-01-09 08:31:44 +00:00
Outdated
Review
Copy Link

Minúscula

Minúscula
alexm commented 2023-01-23 06:46:36 +00:00
Outdated
Review
Copy Link

Esto no debería estar en el changes también???

Esto no debería estar en el changes también???
`userFk` int(10) unsigned NOT NULL, `userFk` int(10) unsigned NOT NULL,
PRIMARY KEY (`notificationFk`,`userFk`), PRIMARY KEY (`notificationFk`,`userFk`),
KEY `notificationSubscription_ibfk_1` (`notificationFk`),
KEY `notificationSubscription_ibfk_2` (`userFk`), KEY `notificationSubscription_ibfk_2` (`userFk`),
pau marked this conversation as resolved Outdated
alexm commented 2023-01-09 08:31:46 +00:00
Outdated
Review
Copy Link

Minúscula

Minúscula
CONSTRAINT `notificationSubscription_ibfk_1` FOREIGN KEY (`notificationFk`) REFERENCES `notification` (`id`) ON DELETE CASCADE ON UPDATE CASCADE, CONSTRAINT `notificationSubscription_ibfk_1` FOREIGN KEY (`notificationFk`) REFERENCES `notification` (`id`) ON DELETE CASCADE ON UPDATE CASCADE,
pau marked this conversation as resolved Outdated
alexm commented 2023-01-09 08:33:25 +00:00
Outdated
Review
Copy Link

Hace falta añadir KEY notificationSubscription_ibfk_1 (notificationFk) ?

Hace falta añadir `KEY `notificationSubscription_ibfk_1` (`notificationFk`)` ?
CONSTRAINT `notificationSubscription_ibfk_2` FOREIGN KEY (`userFk`) REFERENCES `account`.`user` (`id`) ON DELETE CASCADE ON UPDATE CASCADE CONSTRAINT `notificationSubscription_ibfk_2` FOREIGN KEY (`userFk`) REFERENCES `account`.`user` (`id`) ON DELETE CASCADE ON UPDATE CASCADE