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

Avoid setting state for unmounted component #145

Closed
wants to merge 11 commits into from

Conversation

everdimension
Copy link

No description provided.

katspaugh and others added 11 commits April 26, 2021 19:11
* signer wip

* signer wip

* signer wip

* wip

* da mvp

* dep bump

* Signer wip

* ethers signer

* remove yalc from signer

* remove console.logs

* change ethersprovider version

* add docs

* type fixes

* utils/poll: remove double increment

* dep updatE

* doc fix

* remove jsonrpc error check logic in checkError

* ui: hello world

* safe dev ui: hello world

* pin jest version to the one required by react-scripts

* test script fix

* dep bump

* use jest 26.6.0

* bootstrapping

* wip

* wip

* wip

* dep bump

* wip

* trying to make header work

* got this thing to render

* logo alignment

* a battle with changing default font family in mui

* a battle with font scaling

* header

* header battle continues

* material ui warning fixes

* Fix web3modal

* provider store wip

* provider store wip

* provider details wip

* provider details wip

* wip:

* connected account details

* connect/disconnect provider

* change node version for tesT

* dep update

* dep update

* import casing fix

* remove test

* Eslint fix

* remove unused stuff

* remove unused stuff

* add props to provider
* signer wip

* signer wip

* signer wip

* wip

* da mvp

* dep bump

* Signer wip

* ethers signer

* remove yalc from signer

* remove console.logs

* change ethersprovider version

* add docs

* type fixes

* utils/poll: remove double increment

* dep updatE

* doc fix

* remove jsonrpc error check logic in checkError

* ui: hello world

* safe dev ui: hello world

* pin jest version to the one required by react-scripts

* test script fix

* dep bump

* use jest 26.6.0

* bootstrapping

* wip

* wip

* wip

* dep bump

* wip

* trying to make header work

* got this thing to render

* logo alignment

* a battle with changing default font family in mui

* a battle with font scaling

* header

* header battle continues

* material ui warning fixes

* Fix web3modal

* provider store wip

* provider store wip

* provider details wip

* provider details wip

* wip:

* connected account details

* connect/disconnect provider

* welcome page layout

* welcome page layout

* welcome page layout wip

* welcome page layout done

* welcome page connect provider

* deploy contracts wip

* save contract addresses to state

* save contract addresses to localstorage

* deploying safe wip

* deploy safe function

* dep bump

* dep bump

* update package json

* deploy safe and redirect to safe page

* disable btn

* try to fix eslint action

* Remove react-app/jest

* dep update

* remove test
* signer wip

* signer wip

* signer wip

* wip

* da mvp

* dep bump

* Signer wip

* ethers signer

* remove yalc from signer

* remove console.logs

* change ethersprovider version

* add docs

* type fixes

* utils/poll: remove double increment

* dep updatE

* doc fix

* remove jsonrpc error check logic in checkError

* ui: hello world

* safe dev ui: hello world

* pin jest version to the one required by react-scripts

* test script fix

* dep bump

* use jest 26.6.0

* bootstrapping

* wip

* wip

* wip

* dep bump

* wip

* trying to make header work

* got this thing to render

* logo alignment

* a battle with changing default font family in mui

* a battle with font scaling

* header

* header battle continues

* material ui warning fixes

* Fix web3modal

* provider store wip

* provider store wip

* provider details wip

* provider details wip

* wip:

* connected account details

* connect/disconnect provider

* welcome page layout

* welcome page layout

* welcome page layout wip

* welcome page layout done

* welcome page connect provider

* deploy contracts wip

* save contract addresses to state

* save contract addresses to localstorage

* deploying safe wip

* deploy safe function

* dep bump

* dep bump

* update package json

* deploy safe and redirect to safe page

* disable btn

* safe page wip

* type fixes

* cache provider

* sidebar wip

* sidebar wip

* mui theme fixes, sidebar wip

* sidebar wip

* redirects and stuff

* use borderbox box sizing

* loading app

* dep bump

* wip

* display safe app in an iframe

* safe app connector

* connector wip

* getSafeInfo handler

* Handle rpc calls

* fix eslint errors

* regen yarn lock

* fix tsts

* fix tsts

* feedback fixes
* build: remove test files from build outputs

* build: move exception for test files up to root tsconfig.json
)

* make safe-app-provider follow eip1193

* add link to eip1193

* fix

* remove eslint-disable as they're warnings and dont block compilation

* fix types

* Emit events in connect/disconnect

* emit event with a chainid

* dep version bumps

* dont emit disconnect event
@github-actions
Copy link
Contributor

github-actions bot commented Apr 29, 2021

CLA Assistant Lite All Contributors have signed the CLA.

@mmv08
Copy link
Member

mmv08 commented Apr 29, 2021

@everdimension Thank you for the contribution! Could you please sign the CLA? Then we'll be able to merge it.

@everdimension
Copy link
Author

I have read the CLA Document and I hereby sign the CLA

@everdimension
Copy link
Author

recheckcla

@mmv08 mmv08 changed the base branch from master to development April 29, 2021 13:29
@mmv08
Copy link
Member

mmv08 commented Apr 29, 2021

oops, there are a lot of conflicts, I'll just recreate the PR myself checking out from development. Thank you for the contribution!

@mmv08 mmv08 closed this Apr 29, 2021
@github-actions github-actions bot locked and limited conversation to collaborators Apr 29, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants