Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adding Next and Previous Buttons #978

Merged

Conversation

mfixstsci
Copy link
Collaborator

Using django.core.paginator

@pep8speaks
Copy link

pep8speaks commented Jul 10, 2022

Hello @mfixstsci, Thank you for updating !

Cheers ! There are no PEP8 issues in this Pull Request. 🍻

Comment last updated at 2022-09-12 21:06:27 UTC

@bhilbert4 bhilbert4 linked an issue Jul 20, 2022 that may be closed by this pull request
@bhilbert4 bhilbert4 added this to In progress in v1.2.0 via automation Jul 20, 2022
Copy link
Collaborator

@BradleySappington BradleySappington left a comment

Choose a reason for hiding this comment

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

Looks good @mfixstsci

jwql/website/apps/jwql/views.py Outdated Show resolved Hide resolved
jwql/website/apps/jwql/views.py Outdated Show resolved Hide resolved
@mfixstsci
Copy link
Collaborator Author

Looks like I merged the TA monitor, accidentally force pushed, and then re-merged it again.

@mfixstsci mfixstsci merged commit 4f1b939 into spacetelescope:develop Sep 12, 2022
v1.2.0 automation moved this from In progress to Done Sep 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging this pull request may close these issues.

Add Next and Previous buttons to exposure pages
3 participants