@mentioned new user and didn't prompt for email?

blockstack-radiks / banter

A simple Twitter-style app to demo Radiks

https://banter.pub

28 stars 8 forks source link

@mentioned new user and didn't prompt for email? #49

Closed stackatron closed 5 years ago

stackatron commented 5 years ago

Screen Shot 2019-03-05 at 7:01 PM

What does this mean? They are already in the DB?

https://staging.banter.pub/

More Info

Source URL: https://staging.banter.pub/

stackatron commented 5 years ago

screen shot 2019-03-05 at 2 45 53 pm

stackatron commented 5 years ago

Is this just the plaintext with the link or is something messed up here?

stackatron commented 5 years ago

@hstove think what is happening is that I previously invited this ID while testing. So they are invited per Radiks but if I mention them nothing happens? Wonder if we can have an exception for this?

Can you please check on: @vader.id.blockstack on stage?

hstove commented 5 years ago

This is likely due to the websocket closing. The fix for this is either:

Accept breakage when they see the 'restart the app' notification
Update the websocket logic to try and re-connect
Update the 'new message created' logic to not use the websocket (requires some refactoring, but probably a good idea regardless)

stackatron commented 5 years ago

Will close for now, we can readdress if we see it again.