Require authorisation to connect twitter handle

rrrene / elixirstatus-web

Community site for Elixir project/blog post/version updates

http://elixirstatus.com

MIT License

282 stars 44 forks source link

Require authorisation to connect twitter handle #51

Closed skade closed 7 years ago

skade commented 7 years ago

This fixes #48.

The flow changes a little: when clicking on "Connect Twitter", the user will be sent to twitter. On callback, the handle is read from the response and added to the user info. Tokens are not saved.

The edit user page changes slightly: it doesn't allow the user to set the handle, but instead just allows to disconnect it again.

Thanks @PragTob for guiding me through the process.

Hanspagh commented 7 years ago

Seems like we pretty much did the same thing, your solutions seems a little more polished, so I will close #50

rrrene commented 7 years ago

@skade @Hanspagh Thanks for taking a stab at this. :+1:

A fixed up variant of this is now live. Please feel free to test. :tada: