Skip to content

Upgrade pagy gem to latest version to 43.2.4#1607

Merged
jmilljr24 merged 2 commits intorubyforgood:mainfrom
sarvaiyanidhi:1587-upgrade-pagy-43
Jan 26, 2026
Merged

Upgrade pagy gem to latest version to 43.2.4#1607
jmilljr24 merged 2 commits intorubyforgood:mainfrom
sarvaiyanidhi:1587-upgrade-pagy-43

Conversation

@sarvaiyanidhi
Copy link
Contributor

🔗 Issue

#1587

✍️ Description

📷 Screenshots/Demos

NA

@sarvaiyanidhi
Copy link
Contributor Author

We don't have many custom implementations so upgrading was quite easy with minimal changes as mentioned in upgrade guide

  • As we used only one extra for bootstrap in pagy.rb so just had to remove that as it is now part of core and update helper from pagy_bootstrap_nav(@pagy, ...) to @pagy.series_nav(:bootstrap, ...)
  • Update in controller from include Pagy::Backend to include Pagy::Method as per new guide

Copy link
Collaborator

@jmilljr24 jmilljr24 left a comment

Choose a reason for hiding this comment

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

@sarvaiyanidhi Thank you so much for taking care of this!!

@jmilljr24 jmilljr24 merged commit 51b1c42 into rubyforgood:main Jan 26, 2026
5 checks passed
@jmilljr24
Copy link
Collaborator

@sarvaiyanidhi P.S. If you are interested in another Ruby For Good project, feel free to take a look at A Window Between Wolrds. It was a legacy rails app that we just brought up to Rails 8. We just launched the new site to production this month and have a steady flow on new issues right now from user feedback as well as thing we need to build out still.

You've done so many great contributions. We truly appreciate it!

@sarvaiyanidhi
Copy link
Contributor Author

Thanks @jmilljr24 for the kind words. Will definitely check out A Window between worlds and love to contribute in it. Always fan of Ruby For Good projects.

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.

2 participants