-
Notifications
You must be signed in to change notification settings - Fork 80
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
New DAOcreator UI Embedded #1506
Conversation
dOrgJelli
commented
Mar 13, 2020
9a00822
to
807c32d
Compare
807c32d
to
d459ee2
Compare
a3fd4e5
to
b80ca40
Compare
b80ca40
to
712b522
Compare
@cbrzn travis fail :( |
@dOrgJelli is that only a UI update ? |
@tibetsprague @dkent600 any idea why test fail ? |
Looks good to me. The one small change I suggest before merging is to give at least an estimate of how many transactions to wait for once you've started "installing" the organization. |
Created an issue for this here. For now we can just add the following warning:
What do you think @cbrzn @dOrgJelli ? |
@dOrgJelli Fix the merge conflicts and I'll merge this. |
@dkent600 merged |
@EzraWeller @orishim left a comment on the issue Ori created. |