Skip to content

Update scroll limit message for non-logged-in users #261

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

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

pandablue0809
Copy link
Contributor

Issue

Add text to NFT Explorer/NFT Sales pages

Requrements

Instead of "Loading more data is available to logged-in Bithomp Pro subscribers.": Use different filter options or select other search parameters to explore more NFTs that match your interests. Log in to Bithomp Pro to enable infinite scroll and access all existing results!

Implemented

The text was added/updated only on these pages:
https://bithomp.com/en/nft-explorer
https://bithomp.com/en/nft-sales

@ihomp ihomp self-requested a review April 22, 2025 11:40
Copy link
Member

@ihomp ihomp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • That component is used on many pages; we shouldn't change how it behaves there. We need to change the message ONLY for the two pages listed in the ticket.
    Reasons:
  • Some pages don't have search parameters ,so that message wouldn't make sense.
  • Some pages are not related to NFTs, like on AMM pool pages. So that wouldn't make sense there, either.

@pandablue0809
Copy link
Contributor Author

hi @ihomp
I fixed
pzl review again

@pandablue0809 pandablue0809 requested a review from ihomp April 22, 2025 12:27
Copy link
Member

@ihomp ihomp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We should use the same component InfiniteScrolling its just need to have an extra param "noSessionTokenMessage", if it's not present as param the default message should be shown.

@ihomp
Copy link
Member

ihomp commented Apr 24, 2025

It would also be good to update the nft-sales page to use the InfiniteScrolling component.

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.

2 participants