[Add App] Sky.money

[zdumitru]

Entry type

• New addition

App info

URL: https://app.sky.money/

Manifest.json URL: https://app.sky.money/manifest.json

Name: sky.money

Description: Rewards, savings, upgrade, and trade

Icon (PNG, 180x180):

It's minified via https://tinypng.com: yes

Homepage: https://sky.money/
Twitter: https://x.com/SkyEcosystem
GitHub: https://github.com/skybase-int
Discord: https://discord.gg/skyecosystem

App supports batching multiple transactions via Safe: no

Supported networks

- Mainnet

Revision checks

• Used smart contracts were audited.
• You have implemented the app using the Safe Apps SDK
• Your Safe App includes a manifest.json file at the root with the required data
• The app can be loaded as a custom Safe App in the Apps section of https://app.safe.global.
• The app auto-connects to the Safe as a wallet
• It doesn't try to connect to the browser wallet (e.g. MetaMask)
• You are able to trigger and execute one transaction with a Safe.
• RPC requests are optimized (not triggering many requests in a very short time period).

Audit document

https://www.chainsecurity.com/smart-contract-audit-reports?client=MakerDao

Code for review

https://github.com/skybase-int/webapp

Team information

Company: TechOps Services

Official website: https://techops.services/

Point of contact: Dumitru

Email/Telegram: contact[at]techops.services

[kirkkonen]

This app was reviewed and approved by the product team.

[zdumitru]

This app was reviewed and approved by the product team.

Hi @kirkkonen ,
Thanks for approving the app. Can you please share when the app is going to be live? Currently it's not visible in the gnosis safe app list. Thank you!

[PooyaRaki]

This app was reviewed and approved by the product team.

Hi @kirkkonen , Thanks for approving the app. Can you please share when the app is going to be live? Currently it's not visible in the gnosis safe app list. Thank you!

Hi @zdumitru
Thank you for your submission! The tech team will first review the code and the audit results, followed by the QA team’s review. Once all reviews are complete, we will proceed to list the app.

[iamacook]

@zdumitru, I’ve reviewed the shared repository, and everything looks good so far. However, I’m unable to find the transaction code. If I understand correctly, it’s handled within the @jetstreamgg/widgets package? Could you grant me access to its code so we can continue the review?

[zdumitru]

@zdumitru, I’ve reviewed the shared repository, and everything looks good so far. However, I’m unable to find the transaction code. If I understand correctly, it’s handled within the @jetstreamgg/widgets package? Could you grant me access to its code so we can continue the review?

Read access given. That repo will be open sourced some time soon.

[iamacook]

Despite the lack of test coverage, I did not encounter any critical issues with the submission. I’m happy to pass it on to QA for further review. cc @francovenica

[francovenica]

1 -Minor issue. The "Copy address" button doesn't work. Clicking on it won't save the address in my clipboard

2 - Major issue:
I tried a trade transaction, USDC for DAI, and I was able to sign and execute the approval, but the app "Doesn't notice" that that tx was successfully executed so I'm stuck waiting for the other tx (The trade itself) to show up

I had this same issue trying an upgrade from DAI to USDS; Tx executed fine, but the app gets stuck waiting for the upgrade to finish

3 - Major issue:
Having the approved I refreshed the app so I can try the trade of USDC for DAI. I signed the message successfully, but the trade still fails.

[PooyaRaki]

@zdumitru We are awaiting your response to proceed with the process.

[zdumitru]

@zdumitru We are awaiting your response to proceed with the process.

Thanks. We'll address the issue and fix is shortly. Will come back to you when done.

[PooyaRaki]

@zdumitru Thanks for the update, I’m closing this issue for now.
Feel free to reopen it with an update whenever the fix is ready.