-
Notifications
You must be signed in to change notification settings - Fork 494
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
chore: upgrade gnosis dependencies and move to peerDependencies #1785
chore: upgrade gnosis dependencies and move to peerDependencies #1785
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
👍 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. |
New dependencies detected. Learn more about Socket for GitHub ↗︎
|
@MeLlamoPablo Awesome! We'll test the sdk part on our end and if it looks good we'll merge, thanks for the work! |
@MeLlamoPablo are there any build env settings with the latest packages that need to be noted here in the documentation with the new deps? |
Hey, this is Yago from Safe. That's okay with us. Would it be difficult for you to update all the "Gnosis" references to "Safe", including the package name? As you probably know, we are now a separate DAO You can remove as well the |
Hey guys, I'm currently OOO, just letting you know that I'll take a look as soon as I can, thanks for your comments! |
I don't think so. Looking through the source code for |
Removed references to the domain on 4a11e18 and updated docs and examples on 3e1b9bf. Renaming the package itself is something that the Blocknative team should do. |
@MeLlamoPablo Is this ready for another review? thanks! |
Yeah go ahead!
El 21 de julio de 2023 21:49:33 CEST, Taylor Dawson ***@***.***> escribió:
…
@MeLlamoPablo Is this ready for another review? thanks!
--
Reply to this email directly or view it on GitHub:
#1785 (comment)
You are receiving this because you were mentioned.
Message ID: ***@***.***>
|
Hey @MeLlamoPablo thanks for the renaming !! ... really appreciate |
@MeLlamoPablo so you want the name of the package to be changed to Safe? |
I think the package should eventually be renamed to |
Opened an issue for you regarding the renaming #1849. Thanks! |
@MeLlamoPablo Once the merge conflicts are resolved this looks good to go 🙌 |
Solved! |
Description
This PR upgrades
@safe-global/safe-apps-sdk
(f.k.a.@gnosis.pm/safe-apps-sdk
) to8.0.0
and@safe-global/safe-apps-provider
(f.k.a.@gnosis.pm/safe-apps-provider
) to0.17.1
.It also moves them from
dependencies
topeerDependencies
as this Onboard module shouldn't dictate the Safe Apps SDK version a dApp uses. A dApp may want to load the Safe Apps SDK independently of Onboard and thus listing it ondependencies
would result in either two different versions being loaded or the dApp being unable to upgrade the Onboard version.Note that this is a breaking change as the consumers of this package must install
@safe-global/safe-apps-sdk
and@safe-global/safe-apps-provider
independently.PLEASE NOTE- Checklist must be complete prior to review.
Checklist
package.json
of the package you have made changes in following semantic versioning and using alpha release taggingyarn check-all
to confirm there are not any associated errorsDocs Checklist
docs/package.json
file (if applicable) - Not applicableIf 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 athttp://localhost:8080/
Tests with demo app (injected)
Tests with demo app (SDK)
Kindly requesting help with this, as I can't get the dev server running without errors