Workout page in gym needs pagination

[s-edwards]

When viewing the list of workouts in the gym, pagination should be used, just like when viewing the list of exercises in the gym. Currently, the workout index page is unpaginated.

We should just port the pagination approach from the exercise index page over to the workout index page as well.

[i-m-karanagarwal]

@s-edwards I would like to work on this issue.

[s-edwards]

Excellent. We would appreciate the contribution. Right now, you can find pagination support already on the exercises page in the gym, and all that is required is to duplicate that feature on the workouts page. If you aren't sure which views are the two relevant ones to look at, let us know. Thanks!

[i-m-karanagarwal]

@s-edwards I am getting this error while setting up the docker container. How do I solve this?

[s-edwards]

Hmmm ... haven't seen that error yet, no doubt because of the way docker caches the image stack as it builds a new image.

To fix, edit the Gemfile and search for the "github:" clause (it appears on three gems). replace with "git:" and include a full URL instead of just the github user name/repo. For example, try replacing this:

gem 'andand', github: 'raganwald/andand'

With something like this instead:

gem 'andand', git: 'https://github.com/raganwald/andand.git'

Do the same for the other two gems that use "github:", and then you can include the fix as part of your pull request (or even use a second pull request to fix this error!).

For more info, google "gemfile unauthenticated git protocol on port 9418 is no longer supported" and you'll find more info and examples.

[i-m-karanagarwal]

@s-edwards I got another error while trying to run the docker image. How do I solve this?

[s-edwards]

Try executing "docker compose build" to see if that clears it. The error message is being produced because of the earlier bundle install failure message you received, I think--it looks like the docker image just needs to be rebuilt with the Gemfile fix you made. Let us know if that doesn't fix it.

[i-m-karanagarwal]

This time "docker compose build" executed without a single error but when I tried to execute "docker compose up" then the same error prompted. How do I solve that?

[ayaankazerouni]

I think that this is due to the old Gemfile.lock still hanging around, even though the Gemfile has been updated. This can probably be avoided by changing our Dockerfile, but in the meantime, can you try doing:

docker compose run web bundle install // this will update the Gemfile.lock based on the changes you made to Gemfile
docker compose up

[i-m-karanagarwal]

I tried these commands and it solved the error. But now it shows another error. How do I solve this?

[s-edwards]

I think I see the root of the problem in the Dockerfile, so I'm going to see if I can fix it quickly.

[s-edwards]

OK, I was looking at the Dockerfile in a different branch. When I do a fresh pull of the staging branch (after merging your pull request), docker compose up builds and runs fine for me.

Try using git status to see what files are changed locally. You may try checking out staging directly and see if that fixes the issue for you. I think the problem is that somewhere in trying to fix the other errors, your local Gemfile.lock in your repo is out of sync and is causing problems, but pulling a clean copy of Gemfile / Gemfile.lock from staging should work.

[i-m-karanagarwal]

@s-edwards Sorry for the delay, But I tried every possible way and it is still not working. It shows the same error as #208 (comment) this. How do I solve this?

[s-edwards]

Does "git status" show any changes to Gemfile.lock (or Gemfile) compared to what is in the staging branch? If so, use "git checkout -- Gemfile.lock" to overwrite the local copy with what is in staging, then try "docker compose build web" to rebuild the docker image with the Gemfile.lock from staging. That is working on my end and launches correctly. If that does not work, please reply with the output from "git status" so we can see what might be different in your local environment.