lobsters/lobsters

Upvoted Comments page has incorrect pagination

Geschlossen

#1.840 geöffnet am 27.12.2025

Auf GitHub ansehenContributor Guide
 (3 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Ruby (961 Forks)user submission
buggood first issue

Repository-Metriken

Stars
 (4.680 Sterne)
PR-Merge-Metriken
 (Durchschn. Merge 7T 4h) (20 gemergte PRs in 30 T)

Beschreibung

When logged in, in the /upvoted/comments, if you have less than 20 comments upvoted, it still shows the "Page 2" button. When clicked, it goes to page 2 which is empty.

I assume this is a bug as Upvoted Stories (/upvoted/stories) doesn't show this button when you don't have enough stories for another page to show.

Contributor Guide

Research-Richtung
Look at app/controllers/upvoted controller.rb and app/views/upvoted/ directory. The issue is that `current user.upvoted comments.paginate` shows a 'Page 2' button even when there are fewer than 20 comments. Compare with the upvoted stories logic to see how it correctly hides the button. Likely fix: adjust the condition for showing pagination links in the view.
Tech Stack
rubyrailspostgresql
Domain
backend
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Frisch
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
Ruby on Railspagination (will paginate)Lobsters views and controllers
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
80