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

Explorer tabs upgrade #485

Merged
merged 26 commits into from
Feb 11, 2023
Merged

Explorer tabs upgrade #485

merged 26 commits into from
Feb 11, 2023

Conversation

fabiolalombardim
Copy link
Contributor

  • Modified existing "Proposal" tab according to Figma
  • Added new "Config" tab

@netlify
Copy link

netlify bot commented Jan 28, 2023

Deploy Preview for tezos-homebase ready!

Name Link
🔨 Latest commit 3a903de
🔍 Latest deploy log https://app.netlify.com/sites/tezos-homebase/deploys/63e78203aa32140008417357
😎 Deploy Preview https://deploy-preview-485--tezos-homebase.netlify.app/
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@fabiolalombardim fabiolalombardim changed the title [WIP] Explorer tabs upgrade Explorer tabs upgrade Jan 29, 2023
@thenerdcat
Copy link
Contributor

Approved. But on Mobile this section of the Config Page is not fully responsive image.png . We could create a separate task for fixing this.

@thenerdcat
Copy link
Contributor

image.png this is how it looks on mobile. the variables are not visible

Copy link
Collaborator

@Man-Jain Man-Jain left a comment

Choose a reason for hiding this comment

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

Screenshot 2023-01-31 at 4 25 57 AM

This should be "Execute Lambda" - "Execute a Lambda already Installed on DAO"

Copy link
Collaborator

@Man-Jain Man-Jain left a comment

Choose a reason for hiding this comment

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

Screenshot 2023-01-31 at 4 30 15 AM

The formula that we apply when taking input from the user and sending the values in the contract (

const proposalFlush = 2 * info.votingSettings.votingBlocks + info.votingSettings.proposalFlushBlocks
and
const expiryPeriod = proposalFlush + info.votingSettings.proposalExpiryBlocks
) should be reverse engineered so we show the users the same values they put in otherwise they'll be confused.

Copy link
Collaborator

@Man-Jain Man-Jain left a comment

Choose a reason for hiding this comment

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

Screenshot 2023-01-31 at 4 35 24 AM

Should show the governance Token Symbol here at the end.

@fabiolalombardim
Copy link
Contributor Author

Captura de pantalla 2023-01-31 a las 12 55 18

Responsive tables fixed

@EightRice EightRice self-requested a review January 31, 2023 17:42
@netlify
Copy link

netlify bot commented Jan 31, 2023

manank left a comment:

screenshot

This should be 49 and 3, XTZ everywhere should be divided by 10 ** 6

Browser metadata
Path:      /explorer/dao/KT1UCEscNaP7dj6FsKrGezDrxuak8b9ibHxp/config
Browser:   Firefox 109.0 on Mac OS 10.15
Viewport:  1512 x 831 @2x
Language:  en-US
Cookies:   Enabled

Open in BrowserStack

Open Deploy Preview · Mark as Resolved

@Man-Jain
Copy link
Collaborator

Man-Jain commented Jan 31, 2023

screenshot

This should not be %age but should be 20000200000.004 GovernanceTokenSymbol

Browser metadata
Path:      /explorer/dao/KT1UCEscNaP7dj6FsKrGezDrxuak8b9ibHxp/config
Browser:   Firefox 109.0 on Mac OS 10.15
Viewport:  1512 x 831 @2x
Language:  en-US
Cookies:   Enabled

Open in BrowserStack

Open Deploy Preview · Mark as Resolved

@Man-Jain
Copy link
Collaborator

Screenshot 2023-01-31 at 4 30 15 AM

The formula that we apply when taking input from the user and sending the values in the contract (

const proposalFlush = 2 * info.votingSettings.votingBlocks + info.votingSettings.proposalFlushBlocks

and

const expiryPeriod = proposalFlush + info.votingSettings.proposalExpiryBlocks

) should be reverse engineered so we show the users the same values they put in otherwise they'll be confused.

This hasn't been resolved

@Man-Jain
Copy link
Collaborator

Screenshot 2023-01-31 at 4 25 57 AM

This should be "Execute Lambda" - "Execute a Lambda already Installed on DAO"

This also hasn't been resolved

@Man-Jain Man-Jain merged commit c33f88d into develop Feb 11, 2023
@Man-Jain Man-Jain deleted the explorer-tabs-upgrade branch February 11, 2023 14:56
EightRice pushed a commit that referenced this pull request Mar 9, 2023
* Add FA1.2 support and remove banner (#489)

Signed-off-by: Manank Patni <manank321@gmail.com>

* Fix README.md

Signed-off-by: Manank Patni <manank321@gmail.com>

* Explorer tabs upgrade and add config page (#485)

* Squashed commit of the following:

commit b4bb2cc
Author: fabiolalombardim <lombardifabiola@hotmail.com>
Date:   Wed Dec 21 12:16:17 2022 +0100

    internal link to Lite

* title block updated & buttons removed

* table filter

* clean up

* dao info tables

* updated missing field

* removed material ui styles

* clean up

* removed view code button

* responsive tables

* requested changes

* delete console.log

* execute text updated

* formulas added

* Add banner for Humanitarian DAO and remove banner one time limit

Signed-off-by: Manank Patni <manank321@gmail.com>

* DAO Creator updates and improvements (#495)

* Squashed commit of the following:

commit b4bb2cc
Author: fabiolalombardim <lombardifabiola@hotmail.com>
Date:   Wed Dec 21 12:16:17 2022 +0100

    internal link to Lite

* added tooltip for token id

* fixed conditional votting settings depending on network

* estimated blocks

* update initialization of values

* useBlocks

* useBlocks updated

* values updated

---------

Signed-off-by: Manank Patni <manank321@gmail.com>
Co-authored-by: fabiolalombardim <37227394+fabiolalombardim@users.noreply.github.com>
Co-authored-by: Fernando Torres <80982814+thenerdcat@users.noreply.github.com>
Man-Jain added a commit that referenced this pull request Apr 19, 2023
* Merge Develop to Master (#500)

* Add FA1.2 support and remove banner (#489)

Signed-off-by: Manank Patni <manank321@gmail.com>

* Fix README.md

Signed-off-by: Manank Patni <manank321@gmail.com>

* Explorer tabs upgrade and add config page (#485)

* Squashed commit of the following:

commit b4bb2cc
Author: fabiolalombardim <lombardifabiola@hotmail.com>
Date:   Wed Dec 21 12:16:17 2022 +0100

    internal link to Lite

* title block updated & buttons removed

* table filter

* clean up

* dao info tables

* updated missing field

* removed material ui styles

* clean up

* removed view code button

* responsive tables

* requested changes

* delete console.log

* execute text updated

* formulas added

* Add banner for Humanitarian DAO and remove banner one time limit

Signed-off-by: Manank Patni <manank321@gmail.com>

* DAO Creator updates and improvements (#495)

* Squashed commit of the following:

commit b4bb2cc
Author: fabiolalombardim <lombardifabiola@hotmail.com>
Date:   Wed Dec 21 12:16:17 2022 +0100

    internal link to Lite

* added tooltip for token id

* fixed conditional votting settings depending on network

* estimated blocks

* update initialization of values

* useBlocks

* useBlocks updated

* values updated

---------

Signed-off-by: Manank Patni <manank321@gmail.com>
Co-authored-by: fabiolalombardim <37227394+fabiolalombardim@users.noreply.github.com>
Co-authored-by: Fernando Torres <80982814+thenerdcat@users.noreply.github.com>

* Revert "Merge Develop to Master (#500)" (#517)

This reverts commit c04c7d8.

* Upgrade beacon version (#531)

* Upgrade beacon version

Signed-off-by: Manank Patni <manank321@gmail.com>

* Upgrade beacon version

Signed-off-by: Manank Patni <manank321@gmail.com>

* Remove connect on originate and save beacon wallet instance

Signed-off-by: Manank Patni <manank321@gmail.com>

---------

Signed-off-by: Manank Patni <manank321@gmail.com>

* Fix validations

Signed-off-by: Manank Patni <manank321@gmail.com>

* Add decimal validation to block number

Signed-off-by: Manank Patni <manank321@gmail.com>

---------

Signed-off-by: Manank Patni <manank321@gmail.com>
Co-authored-by: fabiolalombardim <37227394+fabiolalombardim@users.noreply.github.com>
Co-authored-by: Fernando Torres <80982814+thenerdcat@users.noreply.github.com>
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.

4 participants