Follow-up from #1653026: [META] Use properly typed values in module configuration.

Problem/Motivation

All integers, Booleans, and even octal numbers in config object files are converted to strings.

Proposed resolution

#1653026: [META] Use properly typed values in module configuration has fixed core, so no need to convert all data types to string anymore.

Remaining tasks

tour.tour.views-ui.yml

User interface changes

NO

API changes

NO

Parent: #1653026: [META] Use properly typed values in module configuration

CommentFileSizeAuthor
#5 2106025-5.patch6.6 KBalexander_danilenko
#3 2106025-3.patch6.71 KBalexander_danilenko
#1 2106025-1.patch8.47 KBalexander_danilenko

Comments

alexander_danilenko’s picture

alexander_danilenko commented
StatusFileSize
new 2106025-1.patch8.47 KB
alexander_danilenko’s picture

alexander_danilenko commented
Status: Active » Needs review
alexander_danilenko’s picture

alexander_danilenko commented
StatusFileSize
new 2106025-3.patch6.71 KB

Wrong understanding of task, my apologies. Correct patch

Status: Needs review » Needs work

The last submitted patch, 2106025-3.patch, failed testing.

alexander_danilenko’s picture

alexander_danilenko commented
StatusFileSize
new 2106025-5.patch6.6 KB
alexander_danilenko’s picture

alexander_danilenko commented
Status: Needs work » Needs review

wrong line endings in patch file, fixed

alexander_danilenko’s picture

alexander_danilenko commented
Assigned: Unassigned » alexander_danilenko
vijaycs85’s picture

vijaycs85
Location London, UK
commented
Issue summary: View changes
Status: Needs review » Needs work 
+++ b/core/modules/views_ui/config/tour.tour.views-ui.yml
@@ -1,24 +1,24 @@
+label: Views ui
...
+    label: Active display
+    body: This is the active display in the view. When there are multiple displays, one link for each display is shown and you can switch displays simply by clicking on the display link.
...
...
...

all tabel/body field with multiple words should be in quotes.

vijaycs85’s picture

vijaycs85
Location London, UK
commented
Status: Needs work » Closed (cannot reproduce)

Just revisited tour.tour.views-ui.yml and current file looks fine. So closing this issue.