Jamal James

Results 219 comments of Jamal James

> @jamaljsr I need your help with (1), since you know what is causing it. I also need your opinion with (4), because I am not familiar with Polar's preferences...

Hey @uwla Thanks for the updates 👌 I will test out these changes in the next day or two. I'm not sure why you're having those issues which I was...

@uwla Apologies on the delayed response. I got tied up with other projects. I'll review this again as soon as I can this week.

Hey @Abdulkbk Thanks for opening the PR. It looks like you're off to a great start. My initial feedback just from looking at your screenshots is: 1. I think we...

Hey @Abdulkbk I want to get this merged. Instead of going back and forth for longer, I just added an extra commit with some minor code formatting changes. I also...

Hey @Extheoisah thanks for opening this draft PR. Please tag me as a reviewer when you are ready for me to take a look at it.

I'm going to hold off on merging this until right before shipping the next release. The removal of older CLN nodes from [nodes.json](https://github.com/jamaljsr/polar/pull/879/files#diff-b46ee14cd28b33d48437eefeaf4095c5989f7bd0fec4997fa343f3af1b121f91L39) would break existing Polar users that could...

Hey @toneloc, the images have already been built and pushed to [Docker Hub](https://hub.docker.com/r/polarlightning/clightning/tags). The instructions for how I build the images are in the [docker readme](https://github.com/jamaljsr/polar/tree/cln-24.02.2/docker#core-lightning). If you want to...

@toneloc Thanks so much for testing and confirming it works for you. That is very helpful. > FYI, I had to remove -`-openssl-legacy-provider` in package.json to get it to work...

Oh interesting. From looking at the code involved in that stacktrace, I suspect that this error is not the root cause, but a result of the chart links object being...