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

Comments

yesct’s picture

yesct commented
Assigned: yesct » Unassigned
Status: Active » Needs review
StatusFileSize
new 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 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 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 🇪🇺🌍
commented

@swentel yep that is definitely related!

tim.plunkett’s picture

tim.plunkett
he/him
Primary language English
Location Philadelphia
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
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
commented
Status: Reviewed & tested by the community » Fixed

Thanks!

Status: Fixed » Closed (fixed)

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