Skip to content
This repository was archived by the owner on Aug 17, 2021. It is now read-only.

ArVerify integration #117

Open
wants to merge 13 commits into
base: master
Choose a base branch
from
Open

ArVerify integration #117

wants to merge 13 commits into from

Conversation

@t8
Copy link
Member

@t8 t8 commented Nov 18, 2020 •
edited
Loading

  • Indication of user wallet being verified
    • Design of this may need work
  • Allow user to verify in Verto if they aren't verified
    • This will require a modal with information about the process and why it's necessary

@t8 t8 requested a review from martonlederer November 18, 2020 05:28
@t8
Copy link
Member Author

t8 commented Nov 18, 2020

Note: Do not merge until first AuthNode for ArVerify is online

@t8 t8 marked this pull request as ready for review November 19, 2020 05:33
@t8
Copy link
Member Author

t8 commented Nov 19, 2020

Ready for review, but still waiting for first node to come online before merging.

martonlederer
martonlederer previously approved these changes Nov 20, 2020
View reviewed changes
Copy link
Member

@martonlederer martonlederer left a comment

Choose a reason for hiding this comment

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

LGTM!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

1 more reviewer

@martonlederer martonlederer martonlederer left review comments

Reviewers whose approvals may not affect merge requirements

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@t8 @martonlederer