Added pagy to user tags

[stacytonui]

Fixes #8426 (<=== Add issue number here)

Make sure these boxes are checked before your pull request (PR) is ready to be reviewed and merged. Thanks!

• PR is descriptively titled 📑 and links the original issue above 🔗
• tests pass -- look for a green checkbox ✔️ a few minutes after opening your PR -- or run tests locally with rake test
• code is in uniquely-named feature branch and has no merge conflicts 📁
• screenshots/GIFs are attached 📎 in case of UI updation
• ask @publiclab/reviewers for help, in a comment below

We're happy to help you get this ready -- don't be afraid to ask for help, and don't be discouraged if your tests fail at first!

If tests do fail, click on the red X to learn why by reading the logs.

Please be sure you've reviewed our contribution guidelines at https://publiclab.org/contributing-to-public-lab-software

Thanks!

[gitpod-io]

[codecov]

Codecov Report

Merging #8635 into main will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main    #8635   +/-   ##
=======================================
  Coverage   81.76%   81.76%           
=======================================
  Files         101      101           
  Lines        5901     5901           
=======================================
  Hits         4825     4825           
  Misses       1076     1076           

Impacted Files	Coverage Δ
app/controllers/user_tags_controller.rb	84.28% <100.00%> (ø)

[stacytonui]

Hello @cesswairimu I found another paginate usage in user tags. Also, I need help in understanding the failed test. Thankyou

[cesswairimu]

Lets try and restart @stacytonui

[gitpod-io]

[RuthNjeri]

Hi @stacytonui the build error is related to the pagination changes.. where pagy is trying to paginate an array collection..for example

      @pagy, @user_tags = pagy(UserTag
        .select('value')
        .where("value LIKE :keyword", keyword: "%#{keyword}%")
        .group(:value)
        .order('value ASC')
        .count('value').to_a, items: 24)

When I tried running your changes on the Gitpod, I encountered the same thing...

I looked at the documentation and pagy uses a different method to paginate an array collection..

I made those changes on your files on Gitpod and the error I encountered was no longer there

  @pagy, @user_tags = pagy_array(UserTag
        .select('value')
        .where("value LIKE :keyword", keyword: "%#{keyword}%")
        .group(:value)
        .order('value ASC')
        .count('value').to_a, items: 24)
    elsif @toggle == "value"
      @pagy, @user_tags = pagy_array(UserTag.group(:value)
        .select('value')
        .order('value ASC')
        .count('value').to_a, items: 24)
    else # @toggle == "uses"
      @pagy, @user_tags = pagy_array(UserTag.group(:value)
        .select('value')
        .order('count_value DESC')
        .count('value').to_a, items: 24)
    end

Note that require 'pagy/extras/array' has already been required in the pagy.rb file, all you need to do is change the methods from pagy to pagy_array

Once you do that the build will stop failing 🎉

[stacytonui]

Wow @RuthNjeri such a lifesaver!..Thank you so much

[codeclimate]

Code Climate has analyzed commit 9fa95a2 and detected 0 issues on this pull request.

View more on Code Climate.

[jywarren]

Wow, tremendous! A tough set of changes and you got it passing the tests!

Many thanks also to @RuthNjeri for the deep research here. Great work both of you!

[jywarren]

Congrats!!!

[ghost]

Nice one @stacytonui @RuthNjeri