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

Add Brick Towers app #404

Open
8 tasks done
skisel-bt opened this issue Oct 22, 2024 · 5 comments
Open
8 tasks done

Add Brick Towers app #404

skisel-bt opened this issue Oct 22, 2024 · 5 comments

Comments

@skisel-bt
Copy link

skisel-bt commented Oct 22, 2024

Entry type

  • New addition

App info

URL:

Manifest.json URL:
Prod (Mainnet): https://safe.bricktowers.io/manifest.json
Staging (Holesky): https://safe.dev.bricktowers.io/manifest.json

Name: Brick Towers Staking

Description: Stake with Brick Towers and enjoy secure native staking or pooled staking options. Maximize rewards with our enterprise-grade security and full custody control. No slashing risks—your assets are always safe.

Icon (PNG, 180x180): https://safe.bricktowers.io/logo180.png
It's minified via https://tinypng.com: yes

Homepage: https://safe.bricktowers.io/
Twitter: https://x.com/BrickTowers
GitHub: https://github.com/bricktowers/

App supports batching multiple transactions via Safe: not needed

Supported networks

- Mainnet
- Holesky

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

The app is mainly a wrapper around Stakewise staking pool contracts: https://github.com/stakewise/contracts/tree/master/audits
There is also custom Ethereum fee splitter used in case of native staking: https://github.com/bricktowers/ethereum-contracts/tree/main/eth-validator-fee-splitter/audit

Team information

Company: Brick Towers AG

Official website: https://bricktowers.io/

Point of contact: Sergey Kisel

Email: [email protected]

Telegram: @sergeykisel

@kirkkonen
Copy link

This submission was reviewed and approved by the product team.

@tmjssz
Copy link

tmjssz commented Nov 19, 2024

This submission is missing a link to the code for review. A link to the production app is also missing.

@skisel-bt
Copy link
Author

@tmjssz link for the app on production is https://safe.bricktowers.io/

The app without safe integration would be on https://staking.bricktowers.io/ or on holesky https://staking.dev.bricktowers.io/

Is the code for the app itself is essential to get through the process? Safe dapp code is part of the bigger frontend application repository (Brick Towers Dashboard), which is currently not open-source. Is there a particular functionality you are interested in, could we share with you some fragments?

@tmjssz
Copy link

tmjssz commented Nov 20, 2024

Hi @skisel-bt the code review is a required part of the process to hightlight potential issues or suggest improvements. We're particularly interested in how the app integrates with Safe. Can you send an invite to the private repo so we can take a look at the code?

@skisel-bt
Copy link
Author

hi @tmjssz the invite has been sent to you to access our repository. Please let me know if it worked out.

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

No branches or pull requests

3 participants