pull down to refresh

178 sats \ 2 replies \ @ek 18 Feb 2023 \ parent \ on: SN bounty: display image when provided an image link meta
I didn't start working on this to give new contributors a chance :) (I have been contributing to the SN codebase for some time now and I also claimed the previous bounty.)
Looking at the comments and Github, I think no one else has picked this up so I think you have good chances to get the bounty.
I am also glad to see someone picking this up!
write
preview
15 sats \ 0 replies \ @jowo 19 Feb 2023
Created a new Github issue for collating any feedback: https://github.com/stackernews/stacker.news/issues/239
reply
0 sats \ 0 replies \ @jowo 18 Feb 2023
thanks for the comment, spent some time tidying up and ended up also implementing the async lookup for when regex won't work.
i'll do a write-up in the morning and share it to get some feedback. once I started digging into this feature I found lots of little edge cases and considered several ways of implementing it, so i'll be curious what you think of the code
reply