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

Update pagination to use params[:_page] per administrate's changes #39

Merged
merged 1 commit into from
May 17, 2021

Conversation

harley
Copy link
Contributor

@harley harley commented Oct 3, 2020

Not sure if gem is still maintained, but here's a quick necessary fix after a new administrate is released.

See thoughtbot/administrate#1725

NOTE: don't release a new version until after administrate 0.15.0 is released, because 0.14.0 doesn't have the mentioned PR yet https://github.com/thoughtbot/administrate/releases/

Relevant excerpt:

Using the param :page for top-level pagination (ie: index pages as opposed to has_many lists in show pages) conflicts with paginating resources whose type happens to be "Page".

See thoughtbot/administrate#1725

Relevant excerpt:

> Using the param :page for top-level pagination (ie: index pages as opposed to has_many lists in show pages) conflicts with paginating resources whose type happens to be "Page".
@SleeplessByte
Copy link
Member

Administrate 14 also doesn't work on Windows :) (fixed in thoughtbot/administrate#1698).

I'll keep an eye out, but shout if they release it?

@harley
Copy link
Contributor Author

harley commented Oct 3, 2020

will do thanks @SleeplessByte !

@SleeplessByte SleeplessByte merged commit 43814ab into XPBytes:master May 17, 2021
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.

None yet

2 participants