It seems that the description of the views are not completely accurate. More of the views says it is for "news" items?

CommentFileSizeAuthor
#4 panopoly_core-viewdescription3-2153941.patch1.11 KBelliotc
#2 panopoly_demo-view-description-2153941.patch518 byteselliotc
#2 panopoly_core-viewdescription2-2153941.patch1.11 KBelliotc
#1 Skærmbillede 2013-12-10 kl. 19.12.38.png66.75 KBlsolesen

Comments

lsolesen’s picture

lsolesen commented
StatusFileSize
new Skærmbillede 2013-12-10 kl. 19.12.38.png66.75 KB
elliotc’s picture

elliotc commented
Component: Code » Admin
Status: Active » Needs review
StatusFileSize
new panopoly_core-viewdescription2-2153941.patch1.11 KB
new panopoly_demo-view-description-2153941.patch518 bytes

Updated the descriptions on 2 of the exported Views.

dsnopek’s picture

dsnopek
he/him
Primary language English
Location USA
commented
Status: Needs review » Needs work

Thanks, @elliotc!

  1. +++ b/panopoly_core.views_default.inc
@@ -12,7 +12,7 @@ function panopoly_core_views_default_views() {
-  $view->description = 'Administrative view to show news.';
+  $view->description = 'View for taxonomy widgets';

    A super small coding standards thing on the panopoly_core patch: The description should end in a period. 

  2. +++ b/panopoly_core.views_default.inc
@@ -140,6 +140,7 @@ function panopoly_core_views_default_views() {
+  $handler->display->display_options['pane_description'] = 'Widget for content matching taxonomy ID';

    This one too. :-)

I know I could change it on commit, but this will be a fun way to practice Drupal.org issue workflow. ;-)

elliotc’s picture

elliotc commented
StatusFileSize
new panopoly_core-viewdescription3-2153941.patch1.11 KB

Cool. Maybe this one will be up to code.

elliotc’s picture

elliotc commented
Status: Needs work » Needs review
dsnopek’s picture

dsnopek
he/him
Primary language English
Location USA
commented
Status: Needs review » Fixed

Looks good! Committed. :-)

Status: Fixed » Closed (fixed)

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