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

Fix relationships pagination repeating None #4870

Merged

Conversation

MrCrayon
Copy link
Collaborator

@MrCrayon MrCrayon commented Apr 6, 2020

None should be added only in the first results page.

Screenshot showing the problem
Schermata da 2020-04-06 06-43-00

This fix was first posted by @howdu that then closed is PR #4690.

Co-Authored-By: howdu howdu@users.noreply.github.com

None should be added only in the first results page.

This fix was first posted by @howdu that then closed is PR thedevdojo#4690.

Co-Authored-By: howdu <howdu@users.noreply.github.com>
@codecov
Copy link

codecov bot commented Apr 6, 2020

Codecov Report

Merging #4870 into 1.4 will not change coverage by %.
The diff coverage is 0.00%.

Impacted file tree graph

@@            Coverage Diff            @@
##                1.4    #4870   +/-   ##
=========================================
  Coverage     63.00%   63.00%           
- Complexity     1369     1370    +1     
=========================================
  Files           194      194           
  Lines          3992     3992           
=========================================
  Hits           2515     2515           
  Misses         1477     1477           
Impacted Files Coverage Δ Complexity Δ
src/Http/Controllers/VoyagerBaseController.php 57.78% <0.00%> (ø) 140.00 <0.00> (+1.00)

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 5892a9c...ef66ee2. Read the comment docs.

@emptynick emptynick merged commit 0c4275c into thedevdojo:1.4 Apr 6, 2020
@MrCrayon MrCrayon deleted the fix-relation-pagination-repeating-none branch May 12, 2020 04:10
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.

None yet

2 participants