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

SocialAuth Integration #54

Merged
merged 5 commits into from
Apr 12, 2023
Merged

SocialAuth Integration #54

merged 5 commits into from
Apr 12, 2023

Conversation

lukeisontheroad
Copy link
Contributor

Proposed changes

Add social auth & signing for Google.
Asana: https://app.asana.com/0/1204165186238194/1204246055017547/f

Types of changes

  • Introduce new account type "SocialAccount" -> account without SK
  • Use Google Login to import a new SocialAccount
  • Use Google Login to sign arbitrary payload

Put an x in the boxes that apply

  • Bugfix
  • New feature
  • Breaking change
  • Documentation Update

Checklist

  • Tested on Mac
  • Tested on Windows
  • Tested on Ubuntu

Further comments

We probably have to transfer the OAuth registration to Trili.

@lukeisontheroad lukeisontheroad marked this pull request as draft April 11, 2023 11:26
@lukeisontheroad lukeisontheroad self-assigned this Apr 11, 2023
@lukeisontheroad lukeisontheroad marked this pull request as ready for review April 11, 2023 12:06
@AndreasGassmann
Copy link
Contributor

I think the package-lock.json can be deleted, because we use yarn?

public/electron.js Outdated Show resolved Hide resolved
public/electron.js Outdated Show resolved Hide resolved
src/setupTests.ts Outdated Show resolved Hide resolved
@leoparis89
Copy link
Contributor

needs a rebase on main

src/types/Account.ts Outdated Show resolved Hide resolved
src/utils/tezos.ts Outdated Show resolved Hide resolved
Copy link
Contributor

Choose a reason for hiding this comment

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

package-lock is not used in the project

@leoparis89
Copy link
Contributor

We need to fix the tests

@leoparis89 leoparis89 self-requested a review April 12, 2023 08:34
@leoparis89 leoparis89 force-pushed the feat/web3-auth branch 3 times, most recently from f4368f1 to 27b0b44 Compare April 12, 2023 16:27
@leoparis89 leoparis89 merged commit 00d2d9f into main Apr 12, 2023
@serjonya-trili serjonya-trili deleted the feat/web3-auth branch April 16, 2023 10:12
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.

None yet

3 participants