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

Filter from relays #63

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

Filter from relays #63

wants to merge 4 commits into from

Conversation

ericspaghetti
Copy link
Collaborator

@ericspaghetti ericspaghetti commented Mar 11, 2024

  • Updates to "Add new listing"
    • Creates g tag for geotags. Relays support filtering/indexing for g tag but not location tag. Filtered queries on marketplace page will be based of iso31661 or other geocodes instead of matching country string names.
    • self mark listing as containing sensitive content label with l tag. Adds Switch component to form.
    • creates tags s based off of listing title and description keywords. this is to index keywords to support searching since NIP-50 is not yet supported by relays for listings.
    • ux updates
  • Marketplace
    • blurs images labeled as sensitive, only works for new listings
    • Filters now query relays directly
    • search and location only work for new listings as tags need to be indexed by relays.
  • My listings
    • show all listings regardless of filters
  • Mobile
    • use floating circle Plus (+) button to add new listing

misc

  • fix multi img uploads on add new listing form

utils/text.ts Fixed Show fixed Hide fixed
utils/text.ts Fixed Show fixed Hide fixed
utils/text.ts Fixed Show fixed Hide fixed
@ericspaghetti ericspaghetti changed the title Filter from relays, UX updates Filter from relays Mar 11, 2024
@thomasyeung687 thomasyeung687 force-pushed the ux-updates-filters branch 2 times, most recently from df8ae0a to dbc8de2 Compare June 19, 2024 19:43
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.

1 participant