This repository has been archived by the owner on Aug 1, 2023. It is now read-only.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
While reviewing the Radiks bug reported from Stackit, I ran Stackit to try and debug the issue. It turns out that the user was never actually being logged in with Radiks. There was some code to call
await User.createWithCurrentUser()
, but that never actually ran. It was inside of ahandleSignIn
function that never got called. Blockstack auth did get called, however, inApp#componentWillMount
. So, I just added the Radiks sign in method there.It's also the same fix from #7. You probably thought it wasn't right because that function was already written, but that block of code never actually runs.
After doing so, I was able to create a new Block, both with and without a collaborator. The home page doesn't actually have UI to display the Blocks, but I assume that's because you all were stuck on this issue.
N.B.: When inviting a user to a
UserGroup
in Radiks, the user has to have signed up for the app before you can invite them. This is a technical limitation, because Radiks needs their public key in order to encrypt an invitation for them.I also recognize that error messages in Radiks are useless. I made an issue for that: stacks-archive/radiks#52