pull down to refresh

0 sats \ 7 replies \ @ek 24 Aug \ parent \ on: 3 Hours Not Using SN Wallet: My Impressions meta
Huh, weird, I tested it yesterday and I saw them getting saved thanks to fa881a1a which is included on master in 789d7626.
write
preview
0 sats \ 6 replies \ @k00b 24 Aug
If you start from scratch, with it detached, not merely resaving an existing one, it fails for me.
reply
0 sats \ 5 replies \ @ek 24 Aug
Mhh, I could swear it worked for me yesterday and at my first attempt today but when I started recording to prove, it's indeed not saving them even though we return the credentials in testSendPayment.
But I found another bug, PR incoming after confirmation.
reply
0 sats \ 4 replies \ @k00b 24 Aug
I think I fixed it. I'll push a pr in a second.
reply
0 sats \ 3 replies \ @k00b 24 Aug
Never mind. It's still broken. Sometimes it works if I hard refresh after detaching for some reason.
reply
0 sats \ 2 replies \ @ek 24 Aug
Seems to work consistently for me after hard refresh, see this video. This must have been what I've seen yesterday and initially today.
My guess would be that detach doesn't completely reset LNC.
reply
0 sats \ 1 reply \ @k00b 24 Aug
It's disconnecting according to lnc-web's api. I suspect lnc-web is holding onto to some state that it shouldn't be.
reply
0 sats \ 0 replies \ @k00b 24 Aug
Poor siggy. I'll send you a message in chat.
reply