Primary keys missing

sarwan_verma → made their first commit to this issue’s fork.

Hi @mizage@gmail.com ,
I have fixed this issue "Primary keys missing" and also created MR,
kindly review the MR.

I reviewed the MR. On the publishing_options_bundles table the better primary key option would on the pubid as it does for the publishing_options table. Updated the MR, please review

Thanks for the quick response! Do we need an update hook for existing installations?

Updated the MR with hook_update that tested locally. Note that if you applied https://git.drupalcode.org/project/pub_options/-/merge_requests/15/diffs... already, it wont have any effects. Or you have to revert it to the earlier and run drush updb.

Hi and thanks for the update. I'll check the code and get it merged in once I'm done testing!

Thanks again for the quick response.

I'm seeing the following when applying the update:

[error]  SQLSTATE[23000]: Integrity constraint violation: 1062 Duplicate entry '1' for key 'PRIMARY': ALTER TABLE "publishing_options_bundles" ADD PRIMARY KEY ("pubid"); Array

geocalleo → closed merge request !16

Thanks @geocalleo.

geocalleo → reopened merge request !16

Hi @MukhtarM, I am trying to merge the updates you put in place. But it looks like the merge request is set for 2.x branch. Can you switch that over to 2.0.0 branch? I've been making release tags on that branch for the latest releases.

Yea sure @geocalleo i will do that

MukhtarM → closed merge request !18

@geocalleo i raised against 2.0.0 now. First i tried to raise in the git UI itself (as there no code change), but it didn't worked. Thats why there is a intermediate MR :)