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.

Sign up for GitHub

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

Jump to bottom

Add pagination to the API #69

Open
Snailedlt opened this issue Jun 9, 2022 · 1 comment
Open

Add pagination to the API #69

Snailedlt opened this issue Jun 9, 2022 · 1 comment
Labels
✨ enhancement New feature or request

Comments

@Snailedlt
Copy link
Member

Currently when using the API, we are getting all the handlers and hub information from the database.

This is not scaleable, and we will soon start to notice lag on the website.

To fix this we need to implement either pagination or caching.
@Snailedlt Snailedlt added the ✨ enhancement New feature or request label Jun 9, 2022
@Snailedlt
Copy link
Member Author

splitscreenme-hub/imports/api/Handlers/server/publications.js

Line 38 in 7f02a9a

limit: Math.min(limit, 500),

splitscreenme-hub/imports/api/Handlers/server/publications.js

Line 180 in 7f02a9a

limit: 500,

Snailedlt added a commit to Snailedlt/splitscreenme-hub that referenced this issue Jun 12, 2022
@Snailedlt
up handler limit from 500 to 600. Related to SplitScreen-Me#69
15ac41f
Snailedlt added a commit that referenced this issue Jun 19, 2022
@Snailedlt
Merge pull request #70 from Snailedlt/feature/increase_harcoded_handl…
11f4878 
…ers_limit

up handler limit from 500 to 600. Related to #69
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
✨ enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@Snailedlt