Our client's replication system requires all tables to have a primary key otherwise replication is not reliable.

CommentFileSizeAuthor
#2 panels-no_primary_key_in-2824380-2.patch945 bytesjrglasgow

Comments

jrglasgow created an issue. See original summary.

jrglasgow’s picture

jrglasgow commented
StatusFileSize
new panels-no_primary_key_in-2824380-2.patch945 bytes

Here is a patch to add the primary key

jrglasgow’s picture

jrglasgow commented
Status: Active » Needs review
joelpittet’s picture

joelpittet
he/him
Primary language English
Location Vancouver
commented
Related issues: +#2479879: Move 'panels_page_allowed_types' variable to a custom table.

This may be reverted... Maybe consider merging with the revert stuff here? #2479879: Move 'panels_page_allowed_types' variable to a custom table.

I need this too, though btw. Can you touch up the coding standards on the comments for this patch? I may RTBC it if we decide to keep them separate

trevorw’s picture

trevorw commented

Need this also, patch here seems to work fine. Looked at #2479879 but got lost in what that's all doing for changes so stuck with this one for now. Will watch issues. Thanks.

  • joelpittet committed 9f04fc1 on 7.x-3.x authored by jrglasgow 
    Issue #2824380 by jrglasgow: no primary key in panels_allowed_types...
joelpittet’s picture

joelpittet
he/him
Primary language English
Location Vancouver
commented
Status: Needs review » Fixed

I know the table isn't being used ATM but I committed this because the table is still installed.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.

joelpittet’s picture

joelpittet
he/him
Primary language English
Location Vancouver
commented
Parent issue: » #3157069: Plan for Panels 7.x-3.11

Adding to plan #3157069: Plan for Panels 7.x-3.11