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

Add pagination for satellite selection #4

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

jose1984
Copy link

This pull request introduces a pagination feature to address the limitation of the previous implementation, where only the first 10 elements from the API were displayed in the satellite selection list.

The query has been enhanced to utilize the offset feature as outlined in the API documentation:

limit/x(,x?)

Specifies the number of records to return. Takes an integer for an argument, with an optional second argument to specify offset. For example, /limit/10/ will return the first 10 records; /limit/10,5/ will show 10 records starting after record #5 (rows 6 through 15)

Further details in API docs

Next steps
Please be aware that certain newly added satellites appearing in the results are causing parsing errors due to their structure.

@jose1984 jose1984 force-pushed the feat/add-pagination-to-select-satellite branch from 3e321b1 to 0a9cb73 Compare March 15, 2024 16:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant