This repository was archived by the owner on Dec 11, 2024. It is now read-only.
Release/4.2 #318
Merged
Release/4.2 #318
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
chore: Label all issues in repo as Applications so they show in zenhub
add passed networks as a return value of useContractKit replace NetWorkNames Enum with POJO so that its extendable Make Network interface support arbitrary networks and include option to pass in the nativeCurrency which is used when adding to metamask. For non celo networks dont try to add celo tokens to metamask
…llets that support alternative gas fee currencies
… currency setting ops + demo it in our example app
FEAT: Allow developers to pass in any networks they wish (and fix feeCurrency)
Append current browser location in Metamask mobile
add testing utils start testing of Provider and hook
remove react-test-render as we dont use and its not cool
Unit Testing + Package Upgrading
Breaking Change: Replace full ContractKit with MiniContractKit
* added search bar * added search bar and optimization * made some changes * feat: initial redesign * chore: remove unused unstated-next * chore: review feedback * fix: bad merge + pr review * perf: search * feat: adjust install logic Co-authored-by: Henry Nunez <henry.nunez@clabs.co>
Fix StableTokenBRL error: Catch token not registered error, display non-deployed token in balances
Add svg icon for celo terminal
* docs(readme): fix typo * docs(readme): add better development instructions * refactor: remove check for window * docs(readme): remove v2 of WC from example * docs(readme): add wallets to list * docs(readme): add supported actions * fix: issue labeler error * fix(docs): original instructions with alternative
fix: remove toggle z-index to not interfere with the modal
* fixes #203 deprecated network prop add defaultNetworkProp allow both for now. and favor network so that this is a nonbreaking change unless you count that this fixes passing in a unfindable network now throws an error instead of silently defaulting to mainnet
* Persistor and Updator respond to events from Connectors * add persistor test and lint * Better emissions typing * improve typings and add explicit event emitter package ( very small) * add test for walletID (used by getRecent())
* Connect modal disable background scroll * Action modal disable background scroll * Fix linting * Move to helper * Remove unused imports * Fix import order * Use window scroll in useFixedBody * Use document.body.style Co-authored-by: zhaonancy <nancy.zhao@clabs.co>
* Chrome Lighthouse accessibility fixes * Fix URL Co-authored-by: zhaonancy <nancy.zhao@clabs.co>
* Add coinbase wallet support * Add dapp name and icon to coinbase wallet sdk
* feat: logger * test: add logging tests * test: fix test
* 4.0 release (#256) * chore: Label all issues in repo as Applications so they show in zenhub * FEAT: Allow developers to pass in any networks they wish add passed networks as a return value of useContractKit replace NetWorkNames Enum with POJO so that its extendable Make Network interface support arbitrary networks and include option to pass in the nativeCurrency which is used when adding to metamask. For non celo networks dont try to add celo tokens to metamask * fix: update Fee Currency should only be an option for Connectors / wallets that support alternative gas fee currencies * lint extra imports * expose supportsFeeCurrency so devs can use to display conditional fee currency setting ops + demo it in our example app * make it clear why we are returning false * conflict * Append current browser location in Metamask mobile * replacing use of contractkit with minicontractkit * Missing Import * Add optional contractCache builder and memoization * Catch token not registered error, display balances * Fix empty block * Change value to be type string * Log error * upgrade to react 18 add testing utils start testing of Provider and hook * upgrade jest and friends remove react-test-render as we dont use and its not cool * feat: redesign and wc fixes (#169) * added search bar * added search bar and optimization * made some changes * feat: initial redesign * chore: remove unused unstated-next * chore: review feedback * fix: bad merge + pr review * perf: search * feat: adjust install logic Co-authored-by: Henry Nunez <henry.nunez@clabs.co> * Balances has BigNumber value and string error * chore: Add svg icon for celo terminal (#69) Add svg icon for celo terminal * docs: improve readme (#184) * docs(readme): fix typo * docs(readme): add better development instructions * refactor: remove check for window * docs(readme): remove v2 of WC from example * docs(readme): add wallets to list * docs(readme): add supported actions * fix: issue labeler error * fix(docs): original instructions with alternative * fix: remove toggle z-index to not interfere * fix: remove specific actions from wallet (#188) * move to target es6 All major browsers support es6 and this should improve bundle size too. https://www.typescriptlang.org/tsconfig#target * Renames use-contract kit to react-celo 4.0 now requires contractkit 2.0 and greater * fix typo Co-authored-by: Camila Rondinini <crondinini@gmail.com> * Add test for modal * no message * Add basic level testing for private-key and wallet connect connectors. * If the network is not set here than initialise call will just re establish connection to metamask with this.network (probably the one it was constructed with) * fix: adjust styles for mobile and specifically for ios safari (#192) * codeReview from @dckesler * Feat/theming (#198) * feat: initial theming * chore: codereview * test: add updateTheme tests * Add Aliases for old function names (#204) Add Aliases for old function names so its not a pain to upgrade to v4 * feat: wallet test plan page (#205) adds a page to the Example app (hidden in this commit) where the user can go through a set of steps to check how well a wallet interacts with the available actions of the package. * fix: export useContractKit (#215) * Some Network Change Errors (#216) * Add tests for address handling in reducer fix test file being misnamed fix some error messages and code comments to be more clear * fix #214 There was a destructuring error when init was not passed which is legal as its optional * fixes #209 when switching network feeCurrency could be undefined. In that case lets skip setting it. * fix: celo-wallet web connection (#217) * fix: celo-wallet web connection * refactor: rename getDesktopLink->getWebLink * fix: destroy not re-rendering * refactor: deeplinks and weblinks logic * Allow updateTheme to take a null value (#219) * Fix #211 Cleanup Init Errors (#223) * Fix #211 Cleanup Init Errors when going to a new Wallet Screen or closing the modal * lint: remove unused import * fix: handle disconnect events coming from sessionUpdate events * fix: show qrcode for mobile connectors * chore: eslint warnings * feat: show mobile disclaimer on apps with a deeplink * fix dark toggle on example app (#227) * Move the toggle down by the rest of the styling options this way it isnt blocking everything on mobile. also make it a little smaller * update the used by list not to include projects that are dead or use a fork * Change css classes to react-celo, include necessary TW css (#231) * 4.0 Alpha 2 * fix example app dependencies (#233) * fix example app dependencies * attempt fix on deploy * chore: configure test in example app (#234) Sets up tests in the example app, using the configuration from the root (modified only to accept .test files outside of __tests__). * docs: Add Migration Guide for v4 (#235) * Add Guides Co-authored-by: Camila Rondinini <crondinini@gmail.com> * style consistency Co-authored-by: Camila Rondinini <crondinini@gmail.com> * pr review @crondinini: clarify version needed * add proper link to release notes Co-authored-by: Camila Rondinini <crondinini@gmail.com> * v4.0.0-beta.1 (React Celo Beta 1) * 4.0 release Co-authored-by: zhaonancy <nancy.zhao@clabs.co> Co-authored-by: Nancy Zhao <32007540+zhaonancy@users.noreply.github.com> Co-authored-by: Nicolas Brugneaux <nicolas.brugneaux@gmail.com> Co-authored-by: Henry Nunez <henry.nunez@clabs.co> Co-authored-by: J M Rossy <jm.rossy@gmail.com> Co-authored-by: Camila Rondinini <crondinini@gmail.com> Co-authored-by: Daniel Kesler <danielckesler@gmail.com> * Fixes #268 (-webkit-fill-available vs background gradient bug) use react modals custom classname to add the styles to html and body only when our modal is open * Fix (hideFromDefaults option not hiding wallets) (#275) * fixes #274 (hide from defaults not hiding wallets) * return an array of which default providers are being used. rather than reuse all screens, which would have had extra and potentially gotten weird * beta 1 * update example app * measure once cut twice * Fix handling of reactModalProp (#278) * fixes the connect modal not recognizing when it receives style props externally. * ensure out styles are not overwritten. * Remove Extra Global Styles (#282) * remove tailwind base as it is messing with global styles * remove unnessessary css resets for react-celo * fix vertical allignments * beta 2 * release * Release/4.0.1 (#277) * Fixes #268 (-webkit-fill-available vs background gradient bug) use react modals custom classname to add the styles to html and body only when our modal is open * Fix (hideFromDefaults option not hiding wallets) (#275) * fixes #274 (hide from defaults not hiding wallets) * return an array of which default providers are being used. rather than reuse all screens, which would have had extra and potentially gotten weird * Fix handling of reactModalProp (#278) * fixes the connect modal not recognizing when it receives style props externally. * Remove Extra Global Styles (#282) * remove tailwind base as it is messing with global styles * remove unnessessary css resets for react-celo * fix vertical allignments * merge release/4.0.1 into master * fix merge issues * remove duplicated files Co-authored-by: zhaonancy <nancy.zhao@clabs.co> Co-authored-by: Nancy Zhao <32007540+zhaonancy@users.noreply.github.com> Co-authored-by: Nicolas Brugneaux <nicolas.brugneaux@gmail.com> Co-authored-by: Henry Nunez <henry.nunez@clabs.co> Co-authored-by: J M Rossy <jm.rossy@gmail.com> Co-authored-by: Camila Rondinini <crondinini@gmail.com> Co-authored-by: Daniel Kesler <danielckesler@gmail.com>
* fix: regression in priority and style issue * refactor: remove all variable shadowings
Bumps [parse-url](https://github.com/IonicaBizau/parse-url) from 6.0.0 to 6.0.2. - [Release notes](https://github.com/IonicaBizau/parse-url/releases) - [Commits](https://github.com/IonicaBizau/parse-url/commits) --- updated-dependencies: - dependency-name: parse-url dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* All Connectors emit Disconnect Event * Connectors emit connected events with params * make emit protected, ONLY callable by connectors, also fix cew test * switch address change to be emited event * WIP * replace loadConfig with resurector + loadPreviusState Move celo provider helpers to their own files. add listeners * remove persists * fix connector stub * end reducer effecting local storage * fix tests, no local storage * connector listerns must be listening BEFORE initialise is called * remove clearPreviousConfig calls from connectors * split up client settup from fetching URI. allow getURI to be called multiple times without creating new clients * expend test for wallet connect * missed extra import * WIP better netowkr changers * a little cleanup * remove unused code * fix tests * tests pass * wip Wallet connect connector connecting steps, changing networks, disconnecting, and inbetween steps * improve wc, handle CBW disconnect, remove account * update todos * fix lint errors * fix ts build * cleanup * replace console.xxx with our custom logger * self review * clear up verbose test runs * lint up * clean up tests. remove weird exit code * revert * add second test condition * more self review * review (@nicolasbrugneaux) rename destroy to disconnect for linguistic symmetry with connect action * a more reasonable time * unmount in the tests * mocking the balance might be why there is a balance Co-authored-by: Aaron DeRuvo <aaron.deruvo@clabs.co>
* steakwallet -> Omni * OMNI -> Omni
* Chore: Upgrade dependencies and fix issues * Fix: Address flaky tests, remove Node 12 support * Chore: test against node 18 too * fix: remove node-ts which is not ts-node Co-authored-by: Aaron <aaron@clabs.co>
* add walletChainId property to track which chain the wallet is connected to will be null if there is no connected wallet or if wallet does not have a concept of chain separate to the dapp chain (ie for ledger and private key connectors) * feat: add manual networking mode which prevents dapp from doing network switching automatically and from being switched by wallet * repurpose unused second parameter in updateNetwork method to allow devs to specify that intent to just update network of rc/dapp (not of wallet) * add info about manualNetwork mode and walletChainId to the readme
* expose more wallet connect errors thru logger and events Co-authored-by: Daniel Kesler <danielckesler@gmail.com>
…ing (#306) * dont ask metamask to unlock if we are already connected and ressurecting * a bit strict * check that the injected we have has an isConnected Function * lint * avoid needing to coerce Co-authored-by: Daniel Kesler <danielckesler@gmail.com>
move react-modal types from devDeps to deps as they are used in the types for the ConnectModal
* Use CeloSDKs 2.2.0 * fix signTyped Data Example * fix not building * limit files include in npm package to js and d.ts files * release version
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Diff For 4.2.0 Release