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

Bookmarking/Watchlist worker #64

Open
yangwao opened this issue Apr 11, 2023 · 3 comments
Open

Bookmarking/Watchlist worker #64

yangwao opened this issue Apr 11, 2023 · 3 comments
Assignees

Comments

@yangwao
Copy link
Member

yangwao commented Apr 11, 2023

@newraina
Copy link
Member

newraina commented Dec 3, 2023

@yangwao @vikiival May I take on this task? My brief design is:

tech stack
cf-workers + cf-d1

request body

{"uid": "0x*******"}

APIs

  • GET /watchlist/:chain/:type - user's NFT/collection list in the watchlist, includes pagination.
  • GET /watchlist/:chain/:type/:id - has the current user added this entity to the watchlist.
  • POST /watchlist/:chain/:type/:id - add to user's watchlist.
  • DELETE /watchlist/:chain/:type/:id - remove from user's watchlist.

@yangwao
Copy link
Member Author

yangwao commented Dec 3, 2023 via email

@newraina newraina self-assigned this Dec 3, 2023
@vikiival
Copy link
Member

Context

I recall we had designed packs/lists back then using Textile but then we pulled out (don't even recall why) and now it's probably time to recycle ♻️ implementation (if we can) so people can create lists?

This could be used of course in a multidimensional way. I.e. ppl sorting stuff like on GoodReads, i.e. Twitter lists are working, i.e. 📂 folders

designed packs/lists back then using Textile

Basically every folder/list something needs to have two entities.

MainEntity - packs / lists / folders
ItemEntity - pack_items ...

Where MainEntity needs to have fields like who, name, other meta (created, updated, amount of items, etc, chain)

ItemEntity has foreign key to the MainEntity and other fields (type) meta (created, updated)

At current state #227 is implemented incorrectly .)

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