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

[WIP] Signal Scheme Alchemy Integration #1504

Closed
wants to merge 44 commits into from

Conversation

mrrobot16
Copy link
Contributor

@mrrobot16 mrrobot16 commented Mar 12, 2020

The purpose of this PR is to have enable to DAO's to have off chain data that can be changed/modified thru proposals.

PR Specs can be viewed here https://hackmd.io/Uf_6V3JTQRuu_Vc1EsJhsg

This PR is try to make this mock up happen:
DAOHeader

jellegerbrandy and others added 30 commits March 2, 2020 11:53
* open proposal in new tab with ctrl-click

* typo

* handle mac command key
Share common modal styles across all modals.
All ModalPopups can be closed by pressing esc or clicking on the background
All modals have a header, body and optional footer
Also move all scheme utilities into a new lib/schemeUtils.ts file
@orenyodfat
Copy link
Contributor

@mrrobot16 travis fail

Copy link
Contributor

@tibetsprague tibetsprague left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

would love to get an overview of how close this is to ready and what is needed to do still?

daoImg: "/assets/images/generic-user-avatar.png",
reputationHolders: memberCount,
holdings: [
{ name: "GEN", amount: 0.24653 },
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

where are these numbers coming from?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry super late response. I had to take time off due to personal issues. I am back now.

What I am currently working on is on making it responsive and render "dao members count" and "dao holdings".

Another todo is when we have a signal scheme proposal we want to be able to show in the proposal summary what off chain data are we modifying so that users know what to expect to change if a signal scheme proposal is passed.

src/components/Dao/DaoHeader.tsx Show resolved Hide resolved
src/components/Dao/DaoHeader.tsx Show resolved Hide resolved
@mrrobot16 mrrobot16 changed the title Signal Scheme Alchemy Integration [WIP] Signal Scheme Alchemy Integration Mar 17, 2020
@orenyodfat orenyodfat temporarily deployed to alchemy-feat-signal-sch-1lceuv March 19, 2020 17:24 Inactive
@dkent600 dkent600 temporarily deployed to alchemy-feat-signal-sch-aj9zep March 31, 2020 10:37 Inactive
@dkent600
Copy link
Contributor

dkent600 commented Apr 6, 2020

@mrrobot16 Can you please fill in the PR description to describe clearly and precisely what is the intent of this PR? Also be sure the title is consistent with the description. Thanks!

If this is really limited to adding a DAOHeader, be clear about that. If it provides more general support for Signal, would like to know. :-)

Also, how and why are you using GenericScheme?

@mrrobot16
Copy link
Contributor Author

mrrobot16 commented Apr 8, 2020

@mrrobot16 Can you please fill in the PR description to describe clearly and precisely what is the intent of this PR? Also be sure the title is consistent with the description. Thanks!

If this is really limited to adding a DAOHeader, be clear about that. If it provides more general support for Signal, would like to know. :-)

Also, how and why are you using GenericScheme?

@dkent600 Thanks for taking time to look into this :)

I have added a description at the beginning of this PR #1504 (comment)

There you will find a spec document https://hackmd.io/Uf_6V3JTQRuu_Vc1EsJhsg and a mockup of what we want to display in the UI.

To anwser your question as far as why we are using GenericScheme. I would not be able to answer since those decisions where not made by me and unfortunately the developer who was working on that part is no longer working on it :/

The backend part of this PR can be found here daostack/subgraph#494

@mrrobot16
Copy link
Contributor Author

Fix there where many conflicts and the Alchemy has changed since this PR was created I have moved this PR #1758

@mrrobot16 mrrobot16 closed this May 27, 2020
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

Successfully merging this pull request may close these issues.

6 participants