Closed devYonz closed 8 months ago
this needs the final commit from my branch
@yfeleke @getify I'm bit confused by the multiple PRs opened and then closed right. Was this by accident or would you be interested to submit them?
@nikgraf apologies, we thought the code in the PR was correct and ready, but further testing showed us it was not working yet.
seems there were some issues with the cpp code, specifically, and return values being empty.
once that's fixed, will re-submit a PR for review/feedback.
Like Kyle mentioned, we are working on the @getify fork to fix bugs and get ed25591 encryption working. The couple from the weekend were caused because GitHub defaults to the original branch when you try to create a PR for a fork. It is a pretty bad design and I've fallen for it twice this weekend.
In other words, I check out the @getify and push a branch to create a PR to our fork. It defaults to the main repo 😣.
I apologize for what must be an annoying number of GitHub emails. I'll shoot you an email when we fire off the finished PR.
@yfeleke @getify btw was the Local first authentication talk recorded yesterday? Is your work related? If you es you might like https://github.com/serenity-kit/opaque & https://github.com/serenity-kit/react-native-opaque (which I think has quite a potential for local first authentication with e2e encryption)
Yes it was recorded, should be out soon. :)
We were working on getting the web demo ported to show off the same concepts in a react-native mobile app. Unfortunately we didn't quite get that working for the talk, but we're still hopeful to have that working soon, and then we'll post a PR back here with the updated code. :)
Hey @nikgraf, will give those a look. Are you interested in helping out with this effort? DM me your discord in LoFi
Recording is up now: https://www.youtube.com/playlist?list=PLTbD2QA-VMnXFsLbuPGz1H-Najv9MD2-H