reply on: Stacker Saloon \ stacker news

pull down to refresh

114 sats \ 17 replies \ @ek 30 Mar \ parent \ on: Stacker Saloon

fix the money, fix the world!

btw, did you take a look at our good first issues?

92 sats \ 16 replies \ @ek 30 Mar

here is my hand-picked good first issue for you

130 sats \ 15 replies \ @BlokchainB 30 Mar

Hahahaha I have no clue where to start but I’ll give it a shot

77 sats \ 14 replies \ @ek 30 Mar

Start with understanding the problem the ticket is describing

120 sats \ 13 replies \ @BlokchainB 30 Mar

Understanding is one thing fixing it is whole other skill set I am still developing. Like I have no clue where this issue exists in the code base. Plus it’s marked as a ui/UX bug but how do I know it’s not a back end issue?

649 sats \ 12 replies \ @ek 30 Mar

120 sats \ 10 replies \ @BlokchainB 31 Mar

Now I understand the bug the sat counter per zap is missing. It should be before replies but it’s missing on this comment for some odd reason

106 sats \ 9 replies \ @ek 31 Mar

it's because it's pinned, see #707679 (comment link) vs #707679 (direct link)

120 sats \ 7 replies \ @SimpleStacker 31 Mar

yep i noticed pinned comments don't show the amount of zaps, and doesn't even let me zap by clicking the lightning bolt as usual. This should probably be changed.

And @BlokchainB, my suggestion for where to start in the code is to look for the logic of pinned comments (search pinned or pin in Github)... that's where I would start, anyway.