Skip to content

Conversation

@rwagner00
Copy link
Contributor

@rwagner00 rwagner00 commented Sep 19, 2023

Customers who installed the plugin before version 1.2.2 have tables which are missing a Primary key. This PR adds a new WP CLI command which allows them to rectify the issue safely, and then either finalize the changes or roll them back easily.

@rwagner00 rwagner00 requested a review from a team as a code owner September 19, 2023 20:08
@rwagner00 rwagner00 changed the base branch from main to develop September 19, 2023 20:52
Copy link
Member

@pwtyler pwtyler left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • lacking tests is a concern (discussed in slack)
  • Document new behavior in READMEs
  • Update changelog(s) in READMEs
  • Add Upgrade Notice informing of the new behavior to README.txt
  • Major version bump should be included in this PR

@rwagner00
Copy link
Contributor Author

@pwtyler Unfortunately without the composer update, the PHP Unit tests fail on the nightly build:

https://app.circleci.com/pipelines/github/pantheon-systems/wp-native-php-sessions/1535/workflows/a9e7734b-3409-4da0-aaf1-edd5a4c90ddd/jobs/4111

Fatal error: Uncaught Error: Class "wpdb" not found in /home/circleci/pantheon-systems/wp-native-php-sessions/inc/class-session.php:269

rwagner00 and others added 2 commits September 29, 2023 13:12
Co-authored-by: Phil Tyler <phil.tyler@pantheon.io>
rwagner00 and others added 5 commits October 4, 2023 11:31
Co-authored-by: Chris Reynolds <chris.reynolds@pantheon.io>
Co-authored-by: Chris Reynolds <chris.reynolds@pantheon.io>
Co-authored-by: Chris Reynolds <chris.reynolds@pantheon.io>
Ryan Wagner and others added 2 commits October 4, 2023 15:41
@rwagner00 rwagner00 merged commit 60030f5 into develop Oct 4, 2023
@rwagner00 rwagner00 deleted the BUGS-6563-Primary-Key branch October 4, 2023 20:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants