Problem/Motivation

This is in ContentEntityChangedTest:

    // The assertion fails unless https://www.drupal.org/node/2513094 is
    // committed.

The assertion obviously passes, and that issue is fixed.

Steps to reproduce

Proposed resolution

Remove the comment.

Remaining tasks

User interface changes

Introduced terminology

API changes

Data model changes

Release notes snippet

Issue fork drupal-3549023

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

joachim created an issue. See original summary.

scontzen made their first commit to this issue’s fork.

ishani patel’s picture

ishani patel commented
Assigned: Unassigned » ishani patel

ishani patel opened merge request !13336

ishani patel’s picture

ishani patel commented
Assigned: ishani patel » Unassigned
Status: Active » Needs review

Hello, @joachim
I've created MR.
Kindly check.

joachim’s picture

joachim commented
Status: Needs review » Reviewed & tested by the community

Looks good. Thanks!

quietone’s picture

quietone commented

The comment was added in #2453153-141: Node revisions cannot be reverted per translation and the assertion in question was already changed in previous versions of the patch. So, I think this is fine to remove.

  • quietone committed 39235c94 on 11.2.x 
    Issue #3549023 by joachim, ishani patel: remove obsolete comment about...

  • quietone committed 3ce5c201 on 11.x 
    Issue #3549023 by joachim, ishani patel: remove obsolete comment about...
quietone’s picture

quietone commented
Version: 11.x-dev » 11.2.x-dev
Status: Reviewed & tested by the community » Fixed

Committed to 11.2 and 11.2x.

Thanks!

Now that this issue is closed, please review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, please credit people who helped resolve this issue.

Status: Fixed » Closed (fixed)

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

quietone closed merge request !13336