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

NOISSUE - Fix count when search by name is performed #767

Merged
merged 3 commits into from
Jun 24, 2019

Conversation

dborovcanin
Copy link
Collaborator

@dborovcanin dborovcanin commented Jun 20, 2019

What does this do?

This pull request fixes total value in the page response when filter by name is applied.

List any changes that modify/break current functionality

There are no such changes.

Have you included tests for your changes?

Yes.

Did you document any new/modified functionality?

Yes, the docs are updated in #766.

@codecov-io
Copy link

codecov-io commented Jun 20, 2019

Codecov Report

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

Impacted file tree graph

@@          Coverage Diff           @@
##           master    #767   +/-   ##
======================================
  Coverage    85.2%   85.2%           
======================================
  Files          69      69           
  Lines        4630    4630           
======================================
  Hits         3945    3945           
  Misses        445     445           
  Partials      240     240

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 f27cb1d...bb102d9. Read the comment docs.

Signed-off-by: Dušan Borovčanin <dusan.borovcanin@mainflux.com>
Signed-off-by: Dušan Borovčanin <dusan.borovcanin@mainflux.com>
Copy link
Contributor

@manuio manuio left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@nmarcetic
Copy link
Collaborator

@dusanb94 Please update branch

@manuio manuio merged commit aa219b6 into absmach:master Jun 24, 2019
rugwirobaker pushed a commit to rugwirobaker/mainflux that referenced this pull request Jun 26, 2019
* Fix total field when search by name

Signed-off-by: Dušan Borovčanin <dusan.borovcanin@mainflux.com>

* Fix tests to check total count

Signed-off-by: Dušan Borovčanin <dusan.borovcanin@mainflux.com>
@dborovcanin dborovcanin deleted the fix-search branch September 30, 2019 12:44
manuio pushed a commit that referenced this pull request Oct 12, 2020
* Fix total field when search by name

Signed-off-by: Dušan Borovčanin <dusan.borovcanin@mainflux.com>

* Fix tests to check total count

Signed-off-by: Dušan Borovčanin <dusan.borovcanin@mainflux.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants