Updated: Comment #0

Problem/Motivation

#1808248: Add a separate module install/uninstall step to the config import process ensures that the default views configuration has correct "position" values... (some displays have the same value which is very wrong)
Make 1808248 easier to review by splitting off those fixes intto a separate issue (this issue) to reduce the size of 1808248 patch

Proposed resolution

Fix position of default views displays configuration position values.

Remaining tasks

  • Make patch.

User interface changes

No.

API changes

No.

CommentFileSizeAuthor
#1 makedefaultviews.2226823.1.patch6.22 KBYesCT
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

YesCT’s picture

YesCT CreditAttribution: YesCT commented
Assigned: YesCT » Unassigned
Status: Active » Needs review
FileSize
makedefaultviews.2226823.1.patch6.22 KB

made by:

git checkout 8.x
git checkout -b another4
git apply 1808248.102.patch
git add *views.*yml
git status
git diff --staged > makedefaultviews.2226823.1.patch
swentel’s picture

swentel CreditAttribution: swentel commented

Related ? #2153469: View page display getting re-ordered on import, leading config system to believe it hasn't been imported yet

swentel’s picture

swentel CreditAttribution: swentel commented
Related issues: +#2153469: View page display getting re-ordered on import, leading config system to believe it hasn't been imported yet
alexpott’s picture

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

@swentel yep that is definitely related!

tim.plunkett’s picture

tim.plunkett
he/him
Primary language English
Location Philadelphia
CreditAttribution: tim.plunkett commented
Status: Needs review » Reviewed & tested by the community

This dovetails nicely with the other issue, but I think this is fine to go in on its own.

dawehner’s picture

dawehner
Primary language German
CreditAttribution: dawehner commented

Perfect!

  • Commit c9a8e33 on 8.x by catch: 
    Issue #2226823 by YesCT: Make default views displays configuration have...
catch’s picture

catch
he/him
Primary language English
CreditAttribution: catch commented
Status: Reviewed & tested by the community » Fixed

Thanks!

Status: Fixed » Closed (fixed)

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