Problem/Motivation

While working on #2394425: Standard profile config/install needs re-export I discovered that the permissions in user.role.content_editor.yml are indented one space not two. See
Configuration file coding standards

Steps to reproduce

N/A

Proposed resolution

Indent to two spaces

Remaining tasks

Review
Commit

User interface changes

N/A

API changes

N/A

Data model changes

N/A

Release notes snippet

N/A

CommentFileSizeAuthor
#2 3221206-2.patch3.32 KBquietone

Comments

quietone created an issue. See original summary.

quietone’s picture

quietone commented
Issue summary: View changes
Status: Active » Needs review
StatusFileSize
new 3221206-2.patch3.32 KB

Simple.patch.

quietone’s picture

quietone commented
Assigned: quietone » Unassigned
guilhermevp’s picture

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

Nothing much to review on this one, works as intended and complies with standards.

  • effulgentsia committed 37e35de on 9.3.x 
    Issue #3221206 by quietone, guilhermevp: Fix indentation in user.role....
effulgentsia’s picture

effulgentsia commented
Status: Reviewed & tested by the community » Fixed

Thanks. Pushed to 9.3.x.

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
commented

It's upsetting that this issue has landed without checking the major follow-ups, well actually critical follow-ups, i.e. #3221258: Fix content editor role to only have permissions that exist. This should have been marked as a duplicate of that one as that one also fixed the formatting because it was based on an export of the role.

quietone’s picture

quietone commented

@alexpott, sorry, I should have looked for duplicates early. By the time I did it was too late in the day and then when I remembered to look this was committed.

Status: Fixed » Closed (fixed)

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