Skip to content

feat: add collection feat: show user owned tag, support filters and search by owner wallet#157

Merged
beeman merged 4 commits intodevfrom
bhagya/collection-page-ui-changes
Sep 2, 2025
Merged

feat: add collection feat: show user owned tag, support filters and search by owner wallet#157
beeman merged 4 commits intodevfrom
bhagya/collection-page-ui-changes

Conversation

@bhagyamudgal
Copy link
Collaborator

https://cap.link/1m2fdnsbxk16y04

  • Now owned nft show Owned badge
  • Filters work
  • Search by owner wallet works
  • Use nuqs to maintain state of search, owner and filters in url so that its shareable

@nx-cloud
Copy link

nx-cloud bot commented Aug 30, 2025

View your CI Pipeline Execution ↗ for commit fb5d34d

Command Status Duration Result
nx affected --target=build --parallel=3 ✅ Succeeded 43s View ↗
nx affected --target=lint --parallel=3 ✅ Succeeded 32s View ↗
nx affected --target=test --parallel=3 --ci --c... ✅ Succeeded 32s View ↗
nx-cloud record -- pnpm nx format:check ✅ Succeeded 5s View ↗

☁️ Nx Cloud last updated this comment at 2025-08-31 17:30:29 UTC

@bhagyamudgal bhagyamudgal requested a review from beeman August 30, 2025 18:12
@beeman
Copy link
Member

beeman commented Sep 2, 2025

Thanks a lot @bhagyamudgal for this PR. This all looks and works great! Scrolling works smooth! 🧈🧈🧈

One slight nit is that there's a scroll bar on the bottom. Going to merge it through but it would be nice if you can fix that in the next PR. Doesn't need to be a separate one :)

image

@beeman beeman merged commit 4277e78 into dev Sep 2, 2025
9 checks passed
@beeman beeman deleted the bhagya/collection-page-ui-changes branch September 2, 2025 16:52
@bhagyamudgal
Copy link
Collaborator Author

Thanks a lot @bhagyamudgal for this PR. This all looks and works great! Scrolling works smooth! 🧈🧈🧈

One slight nit is that there's a scroll bar on the bottom. Going to merge it through but it would be nice if you can fix that in the next PR. Doesn't need to be a separate one :)

image

Thanks, will fix that in upcoming pr ser

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