Skip to content
This repository has been archived by the owner on Sep 22, 2023. It is now read-only.

Sub status new branch #993

Draft
wants to merge 2 commits into
base: develop
Choose a base branch
from
Draft

Sub status new branch #993

wants to merge 2 commits into from

Conversation

dylanj1752
Copy link
Contributor

🗣 Description

Paginate the subscription status page.

💭 Motivation and context

Make the page more readable when there are many subscriptions running.

🧪 Testing

Tested locally against multiple sets of test data.

✅ Pre-approval checklist

  • This PR has an informative and human-readable title.
  • Changes are limited to a single goal - eschew scope creep!
  • All future TODOs are captured in issues, which are referenced
    in code comments.
  • All relevant type-of-change labels have been added.
  • I have read the CONTRIBUTING document.
  • These code changes follow cisagov code standards.
  • All relevant repo and/or project documentation has been updated
    to reflect the changes in this PR.
  • Tests have been added and/or modified to cover the changes in this PR.
  • All new and existing tests pass.

✅ Pre-merge checklist

  • Revert dependencies to default branches.
  • Finalize version.

✅ Post-merge checklist

  • Create a release.

Copy of the changes made to show the subscription status page as paginated.
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant