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

Fix Safe Summoner App + Create a guide for adding a moloch dao to your existing Safe #517

Open
1 of 7 tasks
skuhlmann opened this issue Aug 11, 2024 · 1 comment · May be fixed by HausDAO/user-guide#20
Open
1 of 7 tasks
Assignees
Labels
bug Something isn't working documentation Improvements or additions to documentation enhancement New feature or request Large

Comments

@skuhlmann
Copy link
Contributor

App

Feature Type

  • New Feature
  • Enhancement

Feature

The Safe Summoner app will no longer load in the Safe UI.

Bug report in discord
https://discord.com/channels/709210493549674598/1082758136093478932/1269193480970833920

Is hosted here:
safe-summon.daohaus.club

Code is here
https://github.com/HausDAO/monorepo/tree/develop/apps/summon-safe

Requirements

  1. Asses why the app won't work and fix the issue. Work with DH core team to redeploy and test it to ensure it's working in the Safe UI again.

  2. Then we need to create a new guide in the user guide that outlines the steps for adding a dao to your safe
    https://guide.daohaus.club/summoner/how-to-summon-a-dao

"How to add DAO governance to an existing safe"

Guides repo
https://github.com/HausDAO/user-guide

Screenshots and Context

No response

@skuhlmann skuhlmann added enhancement New feature or request bug Something isn't working documentation Improvements or additions to documentation Large labels Aug 11, 2024
@santteegt santteegt self-assigned this Oct 16, 2024
@santteegt
Copy link
Contributor

The Safe app is loading through the Safe UI. However, there was an issue when trying to summon a DAO on OP Mainnet as it failed to predict the new Baal address because it wasn't using the correct ModuleProxyFactory contract address. This was solved in #529. @skuhlmann we need to make a new release to reflect these changes in the app.

Now, I'll work on the user guide

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working documentation Improvements or additions to documentation enhancement New feature or request Large
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants