Merge pull request #1007 from yonas/feature/2230-missing-feedback-button

Issue #2230: Fix missing Provide Feedback button
warmshowers · Sep 15, 2016 · ae8ff1e · ae8ff1e
2 parents 59ae087 + 748ea91
commit ae8ff1e
Show file tree

Hide file tree

Showing 7 changed files with 10 additions and 1 deletion.
diff --git a/...l/modules/contrib/prepopulate/LICENSE.txt → ...l/modules/patched/prepopulate/LICENSE.txt b/...l/modules/contrib/prepopulate/LICENSE.txt → ...l/modules/patched/prepopulate/LICENSE.txt
diff --git a/...ll/modules/contrib/prepopulate/README.txt → ...ll/modules/patched/prepopulate/README.txt b/...ll/modules/contrib/prepopulate/README.txt → ...ll/modules/patched/prepopulate/README.txt
diff --git a/...all/modules/contrib/prepopulate/USAGE.txt → ...all/modules/patched/prepopulate/USAGE.txt b/...all/modules/contrib/prepopulate/USAGE.txt → ...all/modules/patched/prepopulate/USAGE.txt
diff --git a/...ules/contrib/prepopulate/prepopulate.info → ...ules/patched/prepopulate/prepopulate.info b/...ules/contrib/prepopulate/prepopulate.info → ...ules/patched/prepopulate/prepopulate.info
diff --git a/...s/contrib/prepopulate/prepopulate.install → ...s/patched/prepopulate/prepopulate.install b/...s/contrib/prepopulate/prepopulate.install → ...s/patched/prepopulate/prepopulate.install
diff --git a/...es/contrib/prepopulate/prepopulate.module → ...es/patched/prepopulate/prepopulate.module b/...es/contrib/prepopulate/prepopulate.module → ...es/patched/prepopulate/prepopulate.module
@@ -56,7 +56,10 @@ function _prepopulate_request_walk(&$form, &$request_slice) {
   $limited_types = array(
     'actions',
     'button',
-    'container',
+    // This is a temporary workaround to prevent the Provide Feedback button
+    // from not appearing on profile pages.
+    // See: https://www.drupal.org/node/2679839#comment-11375243
+    //'container',
     'token',
     'value',
     'hidden',

diff --git a/tests/TODO.md b/tests/TODO.md
@@ -0,0 +1,6 @@
+# Todo Items
+
+## Provide feedback
+
+Provide Feedback is a key site interaction that needs test coverage in the future.
+See [Issue #2230](https://pm.savaslabs.com/issues/2230) and the [Drupal issue queue](https://www.drupal.org/node/2679839) for more details.