Skip to content

ensure draftId is not passed to createElementEditor if it’s no longer applicable #17231

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 4 commits into from
May 7, 2025

Conversation

i-just
Copy link
Contributor

@i-just i-just commented May 7, 2025

Description

Steps to reproduce:

  • create a matrix field in element index view mode; entry type can contain only title
  • create a section with an entry type that uses this matrix field
  • create an entry in this section, add a new nested entry to the matrix field and save that nested entry, but don’t save the root entry
  • click the edit icon for the nested entry - “A server error occurred” notification will show, and the nested entry won’t be loaded in a slideout
  • if you reload the page and try to edit the nested entry again, all will work as expected

The issue was that the draftId from the card of a draft version of the nested entry was lingering behind, causing a response/redirect to be returned, instead of the element.

Related issues

n/a

@i-just i-just requested a review from brandonkelly May 7, 2025 11:38
@brandonkelly brandonkelly merged commit 29e4d64 into 5.x May 7, 2025
@brandonkelly brandonkelly deleted the bugfix/refreshing-instances-and-edit-action branch May 7, 2025 16:23
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