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

Arcana Auth SDK support #1744

Merged
merged 1 commit into from
Aug 2, 2023

Conversation

mmjee
Copy link
Contributor

@mmjee mmjee commented May 26, 2023

Description

Added a package to support the Arcana Auth SDK. See the included documentation for more information.

PLEASE NOTE- Checklist must be complete prior to review.

Checklist

  • Increment the version field in package.json of the package you have made changes in following semantic versioning and using alpha release tagging
  • Check the box that allows repo maintainers to update this PR
  • Test locally to make sure this feature/fix works
  • Run yarn check-all to confirm there are not any associated errors
  • Confirm this PR passes Circle CI checks
  • Add or update relevant information in the documentation

Docs Checklist

  • Include a screenshot of any changes (see docs README on running locally)
  • Add/update the appropriate package README (if applicable)
  • Add/update the related module in the docs demo (if applicable)
  • Add/update the related package in the docs/package.json file (if applicable)

Added documentation for the new package.

Documentation screenshot

If this PR includes changes to add an injected wallet or SDK wallet module:

Please complete the following using the internal demo package.
To run this demo use the command yarn && yarn dev to get the project running at http://localhost:8080/

Tests with demo app (injected) – Not relevant

  • send transaction
  • switch chains
  • sign message
  • sign typed message
  • disconnect

Tests with demo app (SDK)

  • send transaction
  • switch chains
  • sign message
  • sign typed message
  • disconnect – Requires some changes right now

@vercel
Copy link

vercel bot commented May 26, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
web3-onboard-docs ❌ Failed (Inspect) Jun 28, 2023 5:42am

@socket-security
Copy link

socket-security bot commented May 26, 2023

👍 Dependency issues cleared. Learn more about Socket for GitHub ↗︎

This PR previously contained dependency changes with security issues that have been resolved, removed, or ignored.

@mmjee mmjee marked this pull request as ready for review May 26, 2023 06:19
@mmjee
Copy link
Contributor Author

mmjee commented May 26, 2023

So, how is the documentation package supposed to install a unpublished package? Should I split this into two PRs, so that the package can be published first?

docs/package.json Outdated Show resolved Hide resolved
@leightkt
Copy link
Contributor

@mmjee Overall- looking good.
Don't worry about the docs not building- we can check the docs when the alpha package is published on merge into develop.
I tested in the demo and was able to connect, disconnect, sign a message, and sign a typed message. However- I could not send a Goerli transaction- video of my test: https://www.loom.com/share/f95f53bfd01c404e9d816876ec1e1eb6

@mmjee
Copy link
Contributor Author

mmjee commented Jun 1, 2023

@mmjee Overall- looking good. Don't worry about the docs not building- we can check the docs when the alpha package is published on merge into develop. I tested in the demo and was able to connect, disconnect, sign a message, and sign a typed message. However- I could not send a Goerli transaction- video of my test: https://www.loom.com/share/f95f53bfd01c404e9d816876ec1e1eb6

I tried to reproduce the above issue but I couldn't actually get it to switch to Görli because of Infura (and the API key is hardcoded).

image 1

@mmjee mmjee requested a review from leightkt June 2, 2023 13:09
@leightkt
Copy link
Contributor

leightkt commented Jun 2, 2023

you can update the goerli rpc url in packages/demo/src/App.svelte
https://ethereum-goerli.publicnode.com

I'll make a note to update that in our demo

@mmjee
Copy link
Contributor Author

mmjee commented Jun 5, 2023

@leightkt Updated the RPC URL, it should work assuming you have Görli ETH in the address.

@leightkt
Copy link
Contributor

leightkt commented Jun 5, 2023

@mmjee I still get the same 'UNSUPPORTED_OPERATION' error shown in my testing video above (able to switch to goerli, have test ETH in my account)

@socket-security
Copy link

socket-security bot commented Jun 27, 2023

New dependencies detected. Learn more about Socket for GitHub ↗︎

Packages Version New capabilities Transitives Size Publisher
typescript 5.1.6 None +0 40.1 MB typescript-bot
@arcana/auth 1.0.6 network +1 297 kB makyl

@leightkt
Copy link
Contributor

@mmjee moving this to draft until the 'UNSUPPORTED_OPERATION' error when sending a transaction can be address. Thx!

@leightkt leightkt marked this pull request as draft June 27, 2023 17:04
@mmjee
Copy link
Contributor Author

mmjee commented Jun 27, 2023 via email

@mmjee mmjee marked this pull request as ready for review July 25, 2023 07:45
@mmjee
Copy link
Contributor Author

mmjee commented Jul 25, 2023

@leightkt Can you confirm whether the issue above still occurs?

@leightkt
Copy link
Contributor

@mmjee I can no longer connect to the wallet
Screenshot 2023-07-25 at 3 09 14 PM

@mmjee mmjee force-pushed the feature/arcana-auth branch from 0998469 to a736a1a Compare July 26, 2023 06:43
@mmjee
Copy link
Contributor Author

mmjee commented Jul 26, 2023

@mmjee I can no longer connect to the wallet Screenshot 2023-07-25 at 3 09 14 PM

Updated the packages, should be working now. Works locally.

@leightkt
Copy link
Contributor

Screenshot 2023-07-26 at 10 17 12 AM

@mmjee
Copy link
Contributor Author

mmjee commented Jul 27, 2023 via email

@leightkt
Copy link
Contributor

leightkt commented Jul 27, 2023

@mmjee I was able to login only on a browser where I can no other extensions installed. Please make a note in the arcana README that this is expected behavior

Once logged in, I can switch chains, sign a message, sign a typed message, and disconnect the wallet.

I cannot send a Transaction on Goerli:
Screenshot 2023-07-27 at 3 42 08 PM

Screenshot 2023-07-27 at 3 42 11 PM

Screenshot 2023-07-27 at 3 45 00 PM

@mmjee
Copy link
Contributor Author

mmjee commented Aug 1, 2023 via email

@leightkt
Copy link
Contributor

leightkt commented Aug 1, 2023

@mmjee success! Please add a note to the the Arcana readme and docs page that users may need to disable other wallet extensions in order to connect to Arcana.

Also- the Arcana docs page needs to be moved to [..1]arcana to respect alpha order. I can push that change (and handle the renaming of the other docs pages to keep the order) if you want to give me write access.

Once those two things are completed it's good to go!

@shaloo
Copy link

shaloo commented Aug 2, 2023

@mmjee success! Please add a note to the the Arcana readme and docs page that users may need to disable other wallet extensions in order to connect to Arcana.

Also- the Arcana docs page needs to be moved to [..1]arcana to respect alpha order. I can push that change (and handle the renaming of the other docs pages to keep the order) if you want to give me write access.

Once those two things are completed it's good to go!

Got it, fixing docs and readme as suggested above. cc: @mmjee @leightkt

@mmjee mmjee force-pushed the feature/arcana-auth branch from c789e7e to 91401c6 Compare August 2, 2023 12:32
@mmjee
Copy link
Contributor Author

mmjee commented Aug 2, 2023

@leightkt All changes are ready to be reviewed.

@leightkt leightkt merged commit 5aec48b into blocknative:develop Aug 2, 2023
@leightkt
Copy link
Contributor

leightkt commented Aug 2, 2023

@mmjee getting an error on our circleci build- please put up a new PR to fix.

Screenshot 2023-08-02 at 11 48 58 AM

@mmjee
Copy link
Contributor Author

mmjee commented Aug 3, 2023

@leightkt Should be fixed by #1860.

leightkt added a commit that referenced this pull request Aug 3, 2023
* Arcana Auth SDK support (#1744)

* Fix: Aracana Config (#1857)

* add steps

* build staging steps

* update arcana package and yarn (#1858)

* update arcana package and yarn

* fix spacing in config

* try update spacing

* reset spacing on config

* update ci branch for test

* add ts-ignore

* reset config to develop

* add file extension to icon

---------

Co-authored-by: Adam Carpenter <adamcarpenter86@gmail.com>

* Update Arcana Auth SDK version (#1860)

* update arcana/demo packages and root version

* add ts-ignore for arcana

* Feat: Add native Base chain support and to all quickstarts (#1859)

* Add Base chain to all quickstarts

* Bump core package in demo

* update core, react, vue versions

---------

Co-authored-by: Adam Carpenter <adamcarpenter86@gmail.com>
Co-authored-by: mmjee <root@git.maharshi.ninja>
leightkt added a commit that referenced this pull request Aug 3, 2023
* Arcana Auth SDK support (#1744)

* Fix: Aracana Config (#1857)

* add steps

* build staging steps

* update arcana package and yarn (#1858)

* update arcana package and yarn

* fix spacing in config

* try update spacing

* reset spacing on config

* update ci branch for test

* add ts-ignore

* reset config to develop

* add file extension to icon

---------

Co-authored-by: Adam Carpenter <adamcarpenter86@gmail.com>

* Update Arcana Auth SDK version (#1860)

* update arcana/demo packages and root version

* add ts-ignore for arcana

* Feat: Add native Base chain support and to all quickstarts (#1859)

* Add Base chain to all quickstarts

* Bump core package in demo

* update core, react, vue versions

---------

Co-authored-by: Adam Carpenter <adamcarpenter86@gmail.com>
Co-authored-by: mmjee <root@git.maharshi.ninja>
Adamj1232 added a commit that referenced this pull request Aug 3, 2023
* Arcana Auth SDK support (#1744)

* Fix: Aracana Config (#1857)

* add steps

* build staging steps

* update arcana package and yarn (#1858)

* update arcana package and yarn

* fix spacing in config

* try update spacing

* reset spacing on config

* update ci branch for test

* add ts-ignore

* reset config to develop

* add file extension to icon

---------

Co-authored-by: Adam Carpenter <adamcarpenter86@gmail.com>

* Update Arcana Auth SDK version (#1860)

* Feat: Add native Base chain support and to all quickstarts (#1859)

* Add Base chain to all quickstarts

* Bump core package in demo

* Release 2.24.7 (develop) (#1863)

* update arcana/demo packages and root version

* add ts-ignore for arcana

* update core, react, vue versions

---------

Co-authored-by: Adam Carpenter <adamcarpenter86@gmail.com>

* Bump WC deps to the latest (#1866)

* Update docs for Base main (#1868)

---------

Co-authored-by: mmjee <root@git.maharshi.ninja>
Co-authored-by: Kat Leight <33187102+leightkt@users.noreply.github.com>
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.

3 participants