Fix mergeUrlParams encoding query parameters

Before: /-/boards/1/issues.json?page=1&per=50&scope=all&utf8=✓&state=opened&milestone_title=#upcoming After: /-/boards/1/issues.json?page=1&per=50&scope=all&utf8=%E2%9C%93&state=opened&milestone_title=%23upcoming

Fix mergeUrlParams encoding query parameters
b7da615c · Eric Eastwood · 5df1a6d3 · b7da615c
Commit b7da615c authored Dec 16, 2017 by Eric Eastwood
Hide whitespace changes
Inline Side-by-side

Showing with 1 addition and 1 deletion

url_utility.js app/assets/javascripts/lib/utils/url_utility.js +1 -1

No files found.
--- a/app/assets/javascripts/lib/utils/url_utility.js
+++ b/app/assets/javascripts/lib/utils/url_utility.js
@@ -18,7 +18,7 @@ export function getParameterValues(sParam) {
 // @param {String} url
 export function mergeUrlParams(params, url) {
  let newUrl = Object.keys(params).reduce((acc, paramName) => {
-    const paramValue = params[paramName];
+    const paramValue = encodeURIComponent(params[paramName]);
    const pattern = new RegExp(`\\b(${paramName}=).*?(&|$)`);

    if (paramValue === null) {