pull down to refresh

Implement bookmarks on SN
2386 sats \ 16 comments \ @k00b 11 Feb 2023 meta
Basic requirements:
  1. allow bookmarking and removing of a bookmark for any item (comments/posts)
  2. display them in a list for the user sorted by most recent first
Feel free to ask any questions about implementation here. If this goes well, we'll begin doing a lot more of these.
1,000,000 sats paid
write
preview
k00b's bounties
1m sats \ 3 replies \ @ek 12 Feb 2023
Created a PR: https://github.com/stackernews/stacker.news/pull/235
reply
24 sats \ 2 replies \ @pi 12 Feb 2023
@ekzyis - Can you use the more familiar bookmark icon, instead of the star, please?
Examples here: https://www.freeiconspng.com/images/bookmark-icon
I think the star is more for "favourites," whereas bookmarks are for "read later," which is more appropriate for this use case.
Thank you.
reply
115 sats \ 0 replies \ @ek 12 Feb 2023
Done! See here or watch the showcase video.
reply
0 sats \ 0 replies \ @ek 12 Feb 2023
Ah, thx, yeah, can do that
reply
482 sats \ 3 replies \ @deleted231216 11 Feb 2023
I could look at this. A SQL table with user_id, post_url, date + some CRUD for save/remove/view bookmarks could work. Will look at the Stacker.news Github and see how users/posts/pages are structured. I have some experience with Node.js. (Couldn't find any work in January, struggling in February, so a bounty could help a lot.)
reply
9 sats \ 2 replies \ @deleted231216 12 Feb 2023
Canceling this (didn't start development yet), as ekzyis was first
reply
121 sats \ 1 reply \ @ek 12 Feb 2023
Sorry mate, I was already familiar with the code base so I thought I take a look at it 🙈
But please don't be demotivated to still check out the code and mess around with it. Will be useful for further bounties!
You can also check out my PR, maybe my approach sucks haha
reply
15 sats \ 0 replies \ @deleted231216 12 Feb 2023
No worries, was still familiarizing myself with the code and structure. Enjoy the bounty. :)
reply
259 sats \ 1 reply \ @WeAreAllSatoshi 11 Feb 2023
It’s probably just how bounties work, but I’d be pretty frustrated/disappointed if I was one of several people who chose to work on this, only to have someone else get their implementation done and accepted first and mine becomes wasted effort. I guess it’s a learning exercise though.
reply
1 sat \ 0 replies \ @k00b OP 11 Feb 2023
If multiple people achieve the result, I'll happily give a runner up a consolation prize. The alternative is I have to screen you which is frustrating too. Tradeoffs.
reply
10 sats \ 0 replies \ @Busybe3z 12 Feb 2023
What a great community here! 😊
reply
3 sats \ 1 reply \ @TheL0wner 11 Feb 2023
isn't this kind of what the jobs section is for...?
reply
3 sats \ 0 replies \ @k00b OP 11 Feb 2023
Originally no, but they probably belong there too.
reply
0 sats \ 0 replies \ @runningbitcoin 12 Feb 2023
tags would be great as well
reply
0 sats \ 0 replies \ @ThrillerX_ 11 Feb 2023
Yes⚡️
reply
0 sats \ 0 replies \ @bskzgbns 12 Feb 2023
I could look at this. A SQL table with user_id, post_url, date + some CRUD for save/remove/view bookmarks could work. Will look at the Stacker.news Github and see how users/posts/pages are structured. I have some experience with Node.js. (Couldn't find any work in January, struggling in February, so a bounty could help a lot.)
reply