Skip to content
This repository has been archived by the owner on Jun 21, 2023. It is now read-only.

Save the PR list filter settings. #1498

Merged
merged 1 commit into from
Feb 16, 2018
Merged

Conversation

grokys
Copy link
Contributor

@grokys grokys commented Feb 16, 2018

Somewhere along the line, the call to SaveSettings in UpdateFilter got removed, causing the filter settings not to get persisted. This reinstates that call.

Fixes #1494

Somewhere along the line, the call to `SaveSettings` in `UpdateFilter` got removed, causing the filter settings not to get persisted. This reinstates that call.

Fixes #1494
@shana
Copy link
Contributor

shana commented Feb 16, 2018

Looks like it disappeared on commit b929d55, which is a merge commit (PR #1312)

For reference, a good way to find stuff like this is to do git log -G [regex/string to search] -m --patch, it'll output merges that have the regex or string.

@shana shana merged commit 59efadc into master Feb 16, 2018
@shana shana deleted the fixes/1494-save-pr-list-filter branch February 16, 2018 18:28
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants