Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

URL query string parameters to remove from URL are now stored in config file #9985

Closed
wants to merge 1 commit into from

Conversation

mattab
Copy link
Member

@mattab mattab commented Mar 31, 2016

follows up #9789

Since we're adding sid to the list of URL params to be removed from all URLs, and since it is a common URL query string, we need to let users customise the list of URL params to exclude.

in #9889 we will later move these into the UI.

@mattab mattab added the Needs Review PRs that need a code review label Mar 31, 2016
@mattab mattab added this to the 2.16.1 milestone Mar 31, 2016
mattab added a commit that referenced this pull request Mar 31, 2016
@mattab mattab added the duplicate For issues that already existed in our issue tracker and were reported previously. label Mar 31, 2016
@mattab mattab closed this Mar 31, 2016
@mattab mattab deleted the url_query_to_remove_in_config branch April 7, 2016 01:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
duplicate For issues that already existed in our issue tracker and were reported previously. Needs Review PRs that need a code review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant