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

initial dao page #988

Closed
wants to merge 2 commits into from
Closed

Conversation

alvaro-ricotta
Copy link
Contributor

Overview

The DAO page was updated to the new gateway version 2.5, unnecessary texts have been removed.

The backgrounds of the MUI version were globally updated.

The Connect component was updated to match the design style of version 2.5.

We added a new tabHeader component.

We added a new variant to the button component, outline, which will be used in other views.

IMPORTANT:
The detail of the proposals is not updated yet because the connect component is still not functioning. Once the connect button is updated and working, we can update the pending style for the internal tabs.

@@ -16,14 +16,15 @@ limitations under the License. */
import React, { useState } from 'react'
Copy link
Contributor

Choose a reason for hiding this comment

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

Why it's not written as typescript? can you please migrate to typescript and have some basic test around it?

@sk-enya
Copy link
Contributor

sk-enya commented Jul 4, 2023

Can you please update with develop branch?

@sk-enya
Copy link
Contributor

sk-enya commented Jul 4, 2023

Can you please confirm which PR has to be merge?

As I can see same changes on the another one as well.

https://github.com/bobanetwork/boba/pull/1006

@sk-enya
Copy link
Contributor

sk-enya commented Jul 5, 2023

Closing the PR as changes have been already handled in the below.

https://github.com/bobanetwork/boba/pull/1006/

@sk-enya sk-enya closed this Jul 5, 2023
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.

3 participants