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

[HOTFIX] Add a warning when a filter is set when looking at documents and users #505

Merged
merged 9 commits into from
Mar 13, 2019

Conversation

jenow
Copy link
Contributor

@jenow jenow commented Mar 11, 2019

What does this PR do ?

When a filter is set when looking at documents it is not abvious and it happens often for user to think there are missing documents but really they are filtered so I added a warning when a filter is set.

fix #333

Other

Removed the patch rule from codecov as it is useless and run in a forever loop anyway.

Note

I didn't change the version as I am going to do a batch of hotfix before patching this version

@jenow jenow self-assigned this Mar 11, 2019
@codecov-io
Copy link

codecov-io commented Mar 11, 2019

Codecov Report

Merging #505 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #505   +/-   ##
=======================================
  Coverage   84.11%   84.11%           
=======================================
  Files          35       35           
  Lines         793      793           
=======================================
  Hits          667      667           
  Misses        126      126

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0fd9128...3a781e5. Read the comment docs.

@jenow jenow changed the title Add a warning when a filter is set when looking at documents and users [HOTFIX] Add a warning when a filter is set when looking at documents and users Mar 11, 2019
@scottinet scottinet merged commit becc7fe into master Mar 13, 2019
@scottinet scottinet deleted the display-warning-on-active-filters branch March 13, 2019 16:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Show some filters exist even if options are closed
5 participants