Request: restore overflow: :last_page

[n-studio]

The overflow: :last_page has been removed and the replacement suggestion is to raise an error and redirect.

There are some situations where redirection is not possible, in particular when using AJAX/Turbo. And the situation would often happen when updating limit.

The workaround is then:

  begin
      @pagy, @records = pagy(query, limit: params[:limit], raise_range_error: true)
    rescue Pagy::RangeError
      @pagy, @records = pagy(query, limit: params[:limit], page: 1)
      @pagy, @records = pagy(query, limit: params[:limit], page: @pagy.last)
    end

Which is quite inelegant. I believe @pagy, @records = pagy(query, limit: params[:limit], overflow: :last_page) is still relevant.

[ddnexus]

I agree that a simple option would look more elegant in your app code, although less explicit.

Unfortunately, the pagy code would be more complicated, harder to maintain and inelegant than the rescue in your app.

About AJAX and turbo: since you redirect to the same page structure, they should be able to handle it transparently (i.e. no code changes required).

[ddnexus]

I forgot that there's actually is a slightly more elegant way to redirect pagy, shown in the docs here