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

Release list pagination, frontend part #482

Open
undera opened this issue Oct 24, 2023 · 4 comments
Open

Release list pagination, frontend part #482

undera opened this issue Oct 24, 2023 · 4 comments

Comments

@undera
Copy link
Collaborator

undera commented Oct 24, 2023

Relates to #372 and 166.

Due to performance issues, we need to employ the pagination approach for the list of releases displayed. Now that we have migrated to ReactJS, it should be easier to implement.

The frontend side of implementation should be classic pagination at the bottom of the list, and using pagination should cause backend API request with corresponding offset and limit query string parameters added. The place where total number of items is returned from backend should for now be assumed and marked with "TODO", so when backend implements it, the frontend is modified to utilize it.

For now, the backend would just ignore those parameters and will still return full list. The backend part will be done separately.

@sebzz2k2
Copy link

Hii,
I can work on it. But i would need a few pointers as i am new to open source and to this project as a whole. So it would be helpful if anyone can guide me.
Thankyou

@undera
Copy link
Collaborator Author

undera commented Oct 28, 2023

Hii, I can work on it. But i would need a few pointers as i am new to open source and to this project as a whole. So it would be helpful if anyone can guide me. Thankyou

Which pointers do you need? As for general Open Source, you'll need to create a "pull request" on GitHub, there are plenty of guides on Internet how to do it.

@eyalyehia
Copy link

Hey, I would love to try this out. Can it still be assigned to me?

@undera
Copy link
Collaborator Author

undera commented Nov 9, 2023

@eyalyehia Feel free to bring PR for it, I usually don't assign issues. The fact of draft PR created serves as better proof of you working on it.

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

No branches or pull requests

3 participants