Skip to content

Fix kanban set previous neighbor - #13

Merged
paulmskim merged 1 commit into
masterfrom
fix-7104-kanban-set-previous-neighbor
Feb 12, 2026
Merged

Fix kanban set previous neighbor#13
paulmskim merged 1 commit into
masterfrom
fix-7104-kanban-set-previous-neighbor

Conversation

@paulmskim

Copy link
Copy Markdown
Collaborator

Description

This PR fixes setting previous neighbor for the kanban component.

Related Issue

Closes: #7104

Testing

Testing Requirements

  • This needs to be tested in multiple browsers.
  • This needs to be tested with both Gravity and Orbital themes.
  • This needs to be tested with and without AJAX submission.
  • This needs accessibility testing.
  • This needs to be tested with different versions of PHP or database servers.
  • This needs to be tested with different versions of WordPress.
  • None of the above.

Testing Instructions

  • Make movement changes to the kanban cards, ensure the retrieved updated kanban is correct based on the move
  • Ensure the loaded kanban data on refresh is correct based on the previousNeighbor property

QA Tracking Spreadsheet

Screenshots

Checklist

  • I've checked the error log to ensure my code doesn't throw any errors.
  • I've added a changelog entry (if necessary) and updated the version number in 4 places, run npm install and committed package-lock.json (if necessary).
  • I have added a "Documentation Needed" label to this PR (if necessary).

@paulmskim
paulmskim requested a review from speerface February 9, 2026 16:57

@speerface speerface left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Tests are passing, and the switch to a deterministic result vs. the last-key fallback is a smart change, so I think this is good to merge. Let's just keep an eye on things to make sure something isn't borking elsewhere (but that should be pretty obvious in the UI). Thanks Paul!

@paulmskim
paulmskim merged commit 80eae5e into master Feb 12, 2026
@paulmskim
paulmskim deleted the fix-7104-kanban-set-previous-neighbor branch February 12, 2026 05:53
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.

2 participants