1. Mar 26, 2019
  2. Mar 25, 2019
    • Amaury Martiny's avatar
      refactor: Don't call transactionCountOf$ until needed (#414) · 9fb795ce
      Amaury Martiny authored
      * Show parity/light.js's logs too
      
      * Remove txCount from withAccount
      
      * withBalance shows component immeidately
      
      * Play around
      
      * Make it work
      
      * Cleaner code
      
      * Remove 'Loading account tokens...' modal
      
      * Re-order import
      
      * Optimize code
      
      * Fix regression
      
      * Fix typo
      
      * Luke's grumbles
      
      * Put "checking..." when fetching async values
      
      * Show gas error on amount field
      9fb795ce
  3. Mar 21, 2019
    • Axel Chalon's avatar
      Parity Ethereum bundling (#458) · 674e3bf3
      Axel Chalon authored
      
      
      * Parity Ethereum bundling
      
      * Grumbles; update light.js
      
      * Grumbles
      
      * Merge in PR#394 ac-upd-post: refactor sendStore post/postRaw
      
      * Download highest version matching requirement rather than latest beta
      
      * Rename RequireVersion to RequireParityVersion
      
      * Grumbles
      
      * Change Parity version requirement to ~2.4.1
      
      * Fix --no-run-parity with Parity already running
      
      * Grumble
      
      Co-Authored-By: default avataraxelchalon <[email protected]>
      674e3bf3
  4. Mar 12, 2019
  5. Feb 25, 2019
  6. Feb 21, 2019
  7. Feb 20, 2019
  8. Feb 15, 2019
  9. Feb 08, 2019
  10. Feb 07, 2019
    • Luke Schoen's avatar
      refactor: Replace STATUS symbols with status object. Rename health states · 0028f1ce
      Luke Schoen authored
      * Change health states to node-internet, node, and sync
      
      * Change from using STATUS symbol states to using a status object that contains the different states with their status
      
      * Show chain name and percentage in modals too
      0028f1ce
    • Luke Schoen's avatar
    • Luke Schoen's avatar
    • Luke Schoen's avatar
      fix: Add connected-offline state. Wrap individual components with RequireHealthOverlay · f7e110be
      Luke Schoen authored
      Summary of actions performed:
      
      * App.js routes
      	* AccountsList
      		* New functionality - `<RequireHealthOverlay require='connected-offline'>`
      			* Note: we need to be connected even for the AccountsList page since most accounts actually come from parity-ethereum (as pointed out by @amaurymartiny). New functionality (@amaurymartiny proposal) uses additional 'connected-offline' state so if we are connected to the node but not to the internet we can show the AccountsList page
      	* Accounts (token list page)
      		* Retain existing functionality - `<RequireHealthOverlay require='sync'>`
      	* Create Account / Import Account
      		* New functionality - `<RequireHealthOverlay require='connected-offline'>`
      	* Backup Account
      		* New functionality - `<RequireHealthOverlay require='connected-offline'>`
      	* Onboarding
      		* New functionality - no overlay since not wrapping App.js
      	* Search tokens (Whitelist)
      		* Retain existing functionality - `<RequireHealthOverlay require='sync'>`
      * Send.js routes
      	* TxForm, Unlock, TxQrCode, ScanSignedTx, SignedTxSummary
      		* Retain existing functionality - `<RequireHealthOverlay require='sync'>`
      	* Sent
      		* Retain existing functionality by wrapping the individual component instead of App.js with: `<RequireHealthOverlay require='connected' fullscreen>`
      
      * Note: `RequireHealthOverlay` has two states:
      		* `require='connected-offline` no alert/overlay when user connected to a local node but not connected to the internet
      		* `require='connected'` shows full screen alert when the user is not connected to the node and and not connected to the internet
      		* `require='sync'` shows when the user is not sync
      * Note: Only the Sent component has a fullscreen overlay
      * Note: This wraps everything related to eth/token/balance/transfer with `require='sync'`
      * Note: We are not wrapping App.js anymore since when using both `<RequireHealthOverlay require='connected' fullscreen>` in addition to wrapping other components such as Accounts.js with `<RequireHealthOverlay require='connected-offline'>` doesn't allow it. Instead just wrap individual components
      f7e110be
  11. Feb 05, 2019
  12. Feb 04, 2019
  13. Feb 03, 2019
  14. Jan 24, 2019
  15. Jan 23, 2019
  16. Jan 22, 2019
    • Luke Schoen's avatar
      fix: Fixes #361. Restrict user from clicking "Scan" to process tx before gas is calculated · e8ecacac
      Luke Schoen authored
      * Add `isEstimatingTxFee` so we can check if all values including `gas` are available
      without actually having to calculate the tx fee and incorporate into estimateTxFee method
      
      * Display and disable the "Checking..." button if the `isEstimatingTxFee` returns false (i.e. `gas`
      still undefined)
      
      * Note: The bug associated with #361 appears to be Parity Signer-specific. If the user can click "Scan" before the `gas` has been calculated, then when it tries to go to the paritytech/fether/packages/fether-react/src/Send/TxQrCode/TxQrCode.js, where it calls `getRlp()` to get the value for the rlp prop of the QrSigner component, which calls `transactionToRlp(this.tx)` to get the RLP of the unsigned tx that is provided as an argument paritytech/fether/packages/fether-react/src/stores/sendStore.js, it crashes if `this.tx` contains a `gas` property that's undefined.
      So we need to prevent the user from being able to click "Scan" before the `gas` property has been determined.
      We do this by disabling the button and displaying "Checking..." until `isEstimatingTxFee` returns true, and only then
      do we display "Scan" and enable the button
      e8ecacac
    • Luke Schoen's avatar
  17. Jan 21, 2019
  18. Jan 19, 2019
  19. Jan 18, 2019