pull down to refresh
120 sats \ 10 replies \ @BlokchainB 31 Mar \ parent \ on: Stacker Saloon
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
reply
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.reply
Oh @BlokchainB, one tip from one journeyman dev to another! Probably 95% of the time "coding" is not writing code but reading it.
Read the code and follow the logic to understand what's going on. AI tools like ChatGPT can really help here! Copy and paste the code in and ask it to explain to you. Sometimes it gets stuff wrong though, so you gotta keep a critical thinking cap on. But it does a pretty good job most of the time and speeds up your understanding by a lot.
Sometimes to fix a bug it's 99% reading code then changing one line to fix the bug haha
reply
Even getting here is hard. I looked at the repo and was overwhelmed lol like I have no idea where to begin to look for this bug. I saw all the branches (like ek’s lightning branch) and was lost. But thanks for the ChatGPT hint never thought to use it in that way
reply
reply
Hahaha I’ll show you
You probably don't need to pay attention to any of the branches. Just focus on understanding
master
.It is hard, but worth it! ChatGPT can help a lot. Another hint is to tell ChatGPT that it uses
next.js
framework. A lot of it is standard coding patterns for that framework so once you get the hang of it, it becomes a lot easier to understand everything.reply
Okay chatGPT I’m using next.js
Ahh okay
reply