search

tactilenews / 100eyes

Dialogtechnologie für den Pivot to People invented by tactile.news GmbH
MIT License
10 stars 1 forks source link

Fix signal onboarding #1911

Closed mattwr18 closed 3 months ago

mattwr18 commented 3 months ago

Fixes #1908

What changed in this PR and why

We decided to update our Signal onboarding to create a signal_onboarding_token similar to what we do with Telegram. We create the contributor in our database and instruct them to send us a message containing only the token, by which we can identify them. Once we identify them by their token, we can save their uuid and complete their onboarding.

We then send them messages with their uuid and identify them with incoming message by it as well.

mattwr18 commented 3 months ago

@roschaefer this is failing the build, I need to revert some changes to tests that I thought needed to be changed and I also need to write tests for the new onboarding/sending/receiving flow. You could test it out though as I think it works as I imagine it should.