-
Notifications
You must be signed in to change notification settings - Fork 3
Add pagination to message loading #69
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
Conversation
|
Deployed (not pinned) on https://ipfs-2.aleph.im/ipfs/QmdVZqYTmXPtq7RDYfdXfExYHu7wnWw97G1hHR7eHJyJEP |
BjrInt
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This review covers code written on a previous iteration. The changes for this specific feature look ok though.
Why is pagination limited to programs only? All views could benefit from it.
* chore: remove solana connect button by adding a feature flag
|
Deployed (not pinned) on https://ipfs-2.aleph.im/ipfs/QmZX4zkaqUgzthAk2NFtjHSH2GqZj5TPp8JJCpYzLty2dT |
…-account into Feat/myprogram-pagination
|
Deployed (not pinned) on https://ipfs-2.aleph.im/ipfs/Qmd1kNCvYBW3SkDNnBhnZQUcQSeXA3AGL4qzdWHE5e2CDz |
This pull request adds pagination functionality to the message loading section of the program. The program now loads 20 messages per page, and users can navigate through all pages using the pagination component.
Changes Made: