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

Bepro 1916 remove the network selector from the navbar #292

Merged
merged 74 commits into from
Dec 7, 2023

Conversation

vhcsilva
Copy link
Member

No description provided.

@vhcsilva vhcsilva marked this pull request as draft November 23, 2023 10:33
@vhcsilva vhcsilva force-pushed the BEPRO-1916-remove-the-network-selector-from-the-navbar branch 2 times, most recently from 18eb3de to dc4677f Compare November 28, 2023 11:29
@vhcsilva vhcsilva requested a review from moshmage November 29, 2023 20:05
@vhcsilva vhcsilva marked this pull request as ready for review November 29, 2023 20:05
Copy link

@moshmage moshmage left a comment

Choose a reason for hiding this comment

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

note added as comment instead of "start review" -.-

@vhcsilva vhcsilva force-pushed the BEPRO-1916-remove-the-network-selector-from-the-navbar branch from 1ede257 to a9e018f Compare December 4, 2023 16:13
@vhcsilva vhcsilva requested a review from moshmage December 4, 2023 16:54
@vhcsilva vhcsilva force-pushed the BEPRO-1916-remove-the-network-selector-from-the-navbar branch from 4db8f65 to 2764e74 Compare December 5, 2023 12:20
Copy link

@moshmage moshmage left a comment

Choose a reason for hiding this comment

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

lgtm

@vhcsilva vhcsilva merged commit e9c1868 into dev Dec 7, 2023
2 checks passed
@vhcsilva vhcsilva deleted the BEPRO-1916-remove-the-network-selector-from-the-navbar branch December 7, 2023 11:04
vhcsilva added a commit that referenced this pull request Jan 22, 2024
* remove chain selector from navbar and mobile menu

* move files

* move files

* update redirection

* update condition

* remove chain from url

* if has one chain, go to /tasks

* remove chain

* add current chain prop

* remove chain and add more columns

* remove chain

* update current chain

* use current chain

* no need to get from contract, already on db

* change how we update marketplace params on store

* use normal button instead

* task id is enough to identify the task

* fix path

* change how we start service

* remove dead code

* start service when interacting with contract

* add network token association

* use get function from store

* show loading when executing validations

* provide get too

* adjust

* use active marketplace

* check if chain was passed

* if has chain should include closed tasks

* refactor

* compare network address too

* adjust types

* update active marketplace on store

* fix reading of undefined

* remove chain

* check connection

* remove chain

* add network address to state

* use role to check if is registry governor

* start service and update marketplace store

* remove chain

* fix types

* remove wallet condition

* remove unused

* call update balance after get data

* add registry parameters

* reset active step

* add chain parameters

* remove chain

* adjust spacing

* fix reading of undefined

* start service

* change when we set erc20 data

* use getAddress from connection

* remove profile inside marketplace

* only update current network if is step 0

* fix condition

* handle network and marketplace change

* change how we update isGovernor and isCouncil

* remove console.log

* adjust profile path

* adjust redirects

* fix reading of undefined

* adjust redirect

* fix build errors

* refactor contract-button structure

* add tests for contract button component

* add registry variant

* fix build errors

* fix eslint warnings

* fix tests

* increase stale time

* adjust condition

* fixes
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.

2 participants