Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Integrate SteemConnect #25

Merged
merged 5 commits into from
Jun 9, 2018
Merged

Integrate SteemConnect #25

merged 5 commits into from
Jun 9, 2018

Conversation

singhpratyush
Copy link
Member

Fix #3 : Integrate SteemConnet

Changes proposed in this PR -

  • Include SteemConnect SDK for
    • Login
    • Vote
    • Comment
    • Follow and unfollow
  • Do not store posting key in localStorage
  • Use cookies to store auth token from SteemConnect with a proper expiry time.

- Add callback for details from OAuth2
- Add access token to cookies with appropriate expiry
- Add user related details to localStorage
- Fix login flow without posting key
- Use steemconnect for vote, comment and (un)follow
- Remove hapramp backend confirmation
Copy link

@bxute bxute left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Changes are meeting the requirements correctly!
👍🏻

@singhpratyush singhpratyush merged commit 053805c into master Jun 9, 2018
@singhpratyush singhpratyush deleted the integrate-steemconnect branch June 9, 2018 15:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants