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.

Sign up for GitHub

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

Jump to bottom

0055 #4399

Open
wants to merge 1,001 commits into
base: mh/add-feature-status-matrix
Choose a base branch
from
Open

0055 #4399

wants to merge 1,001 commits into from

Conversation

0055iran
Copy link

Closes #ISSUE_NUMBER_GOES_HERE

🎉 🎉 🎉

Description

......

Changes

  • High level
  • changes that
  • you made

Reusable Code/Components

Steps to Test

  1. Steps
  2. to
  3. test

Zeplin Design

Quality Assurance

  • The branch name is in lowercase-kebab-case with no prefix (unless it was created from Clubhouse)
  • The base branch is develop or gau (no nested branches)
  • This is related to a maximum of one Clubhouse story or GitHub issue
  • Types are safe (avoid TypeScript/TSLint features like any and disable, instead use more specific types)
  • If code is copied from existing directories, there is an explanation of why this is necesary in the description/changes, and all copying is done in separate commits to make them easy to filter out
  • If your code adds new text to the UI, the added text is translatable

blurpesec and others added 30 commits March 30, 2021 19:41
@blurpesec
Added our new xdai node to nodes list (MyCryptoHQ#3909)
0ea39ba
@FrederikBolding
Small bug fixes (MyCryptoHQ#3911)
2863727 
* Fix styling issue with HW wallets

* Fix TX status for contract deployment transactions

* Fix tests

* Fix Storyshots missing mock

* Fix summer time issue with storyshots

* Format with date-fns and convert to UTC

* Convert Date.now() mock to UTC

* Try defining timezone in Date.now()
Fix icons in Tx and Confirm receipts (MyCryptoHQ#3910)
7f0cd15 
* Lint TxReceiptTotals

* Simplify Amount

* Correct fontSize for Currency

* Amount determines default fiatColor

* Use Currency to display asset Amount

* Use new Amount props for TxReceiptTotals

* Hide TxReceipt glamour icons on mobile

* Style ConfirmTx

* Remove AmountInput in TokenMigration from Tooltip influence

* Fix tests

* Update comment

* Use tx-sent

* Dedupe svg height declarations

* Correct receipt banner icons

* Use correct  icn-fee

* Update snapshots
Remove UI flash in Suspense (MyCryptoHQ#3913)
ee7029f
@FrederikBolding
Fix storybook snapshot (MyCryptoHQ#3921)
4d9658e
Remove act should not return object error (MyCryptoHQ#3922)
00e292c
@tayvano @FrederikBolding
Update copy that displays immediately after a transaction is sent (My…
1ec96b1 
…CryptoHQ#3924)

* Update en.json

* Update en.json

* Update en.json

* Fix snapshot

Co-authored-by: Frederik Bolding <[email protected]>
Use new api for Amount (MyCryptoHQ#3916)
bafb962 
* Prefer Currency over Amount

- Make Currency ticker optional

* Use redux from txHistory

* Reduce identicon sizes in RecentTransactionList and AccountSummary

* Use asset

* Rename assetValue to text

* Update snapshots

* Fix tests

* Update tests

* Remove seconds from SwapQuote expiry date

* Change default for TimeCountdown

* Revert change of getTxsFromAccount

* Remove unnecessary uuid check

* Correct Accountsummary spacing

* Update snapshots
Optimise dev e2e (MyCryptoHQ#3928)
3bac32d
@GuillaumeRx
Remove Musicoin from networks list (MyCryptoHQ#3927)
ee23f0f 
* Remove Musicoin from Networks/DPaths/Nodes

* Fix generateDefaultValues test assertion
@Mrtenz
Fix token loading issues with non-compliant tokens (MyCryptoHQ#3926)
a44361c 
* Fix token loading issues with non-compliant tokens

* Fix eth-scan incompatibilities
Remove ScreenLock, password and vault (MyCryptoHQ#3923)
63ba7f8
@FrederikBolding
Add Pocket back to list of fallback nodes (MyCryptoHQ#3925)
8bc6781
@EVRICE @FrederikBolding
Add EVRICE network (MyCryptoHQ#3918)
8fb2196 
* Update networks.ts

* Update networks.ts

* Update networks.ts

* Update networks.ts

* Update networks.ts

* Update networks.ts

* Combine existing PRs

* Fix test

* Fix explorer url

* Update networks.ts

* Add DPath

Co-authored-by: Frederik Bolding <[email protected]>
@MrLuit
Update 'new' link from medium to self-hosted blog (MyCryptoHQ#3932)
503446c 
* Update 'new' link from medium to self-hosted blog

* Update snapshot
@FrederikBolding
Speed up test runs (MyCryptoHQ#3935)
e03c0cc 
* Remove max workers flag

* Try maxWorkers=2 for CI

* Add newline
@GuillaumeRx
Use redux-persist migration on settings files (MyCryptoHQ#3937)
a89e54a 
* Use redux-persist migration on settings files

* Stop assuming assets exists on accounts assets

* Remove useless checks and custom migrations on import

* Fix tests

* add verification for foreigner keys in settings file

* remove console.log

* fix test

* clarify canImport

* handle existing rates in settings

* add yarn.lock

* revert yarn.lock changes

* remove canIMport

* fix test
Redirect to dashbaord after demo mode is activated (MyCryptoHQ#3942)
8c945fb
@FrederikBolding
Fix Web3 icons for SignMessage (MyCryptoHQ#3944)
e27d59a
@blurpesec
Enable scanning more addresses (MyCryptoHQ#3835)
ce7a1f0
@409H
Fix broken KB links (MyCryptoHQ#3945)
10922ed 
* Updated KB link

* Updated KB link

* Update snapshots

* Update snapshot
@blurpesec
Fixed Trezor Add-Account Bug (MyCryptoHQ#3946)
ea3aca5 
* resolved styling bug for hardwareui component when text too short

* fixed trezor getMultipleAddresses bug
Preserve event name (MyCryptoHQ#3948)
00af782
Track analytics deactivation (MyCryptoHQ#3949)
9b19f53 
* Track analytics deactivation

* Add comment describing event name

* Track export

* Track import app state
@blurpesec
Update balance in hdwtable to be 5 decimal places (MyCryptoHQ#3947)
142dddf
@wtzb
Update Atheios explorer url (MyCryptoHQ#3951)
ad987d4
@Mrtenz
Fix message signing with WalletConnect (MyCryptoHQ#3882)
196a8f9
@FrederikBolding
Use Ethers FallbackProvider (MyCryptoHQ#3939)
49992d0 
* Use Ethers FallbackProvider

* Add missing mock

* Fix a test that broke randomly
Track network for addAccounts (MyCryptoHQ#3959)
0b5ba7e 
* Track network for addAccounts

* Track setDemoMode

* Fix typo
@FrederikBolding
Swap fixes (MyCryptoHQ#3955)
5e34ef4 
* Disable 0x validation of swaps

* Fix issue where account wouldn't be reset when too low base asset balance

* Fix bug with expiry countdown

* Potential solution with doing some of the gas estimation ourselves

* Simplify base asset detection

* Fix test

* Fix an issue with the latest version of Ethers

* Fix tests

* Remove commented code

* Remove console.log

* Move network selection to state

* Update copy

* Scan tokens when swap finished

* Move some logic from StoreProvider to a saga

* Add test for new saga

* Add test for overwriting existing tx

* Add basic e2e test

* Add test for reverse swap quotes

* Add missing test

* Move selectors to hooks

* Fix typing issues with selectors
FrederikBolding and others added 30 commits December 13, 2021 12:34
@FrederikBolding
Disable Unlock referrals for now (MyCryptoHQ#4232)
04c017c
@FrederikBolding
Small fixes to asset handling (MyCryptoHQ#4231)
97a889a 
* Remove assets from account with excluded assets

* Keep assets with a current balance in LS

* Add test

* Remove unused function
@FrederikBolding
Add Contract info to signing step (MyCryptoHQ#4234)
d40dd70 
* Add TxIntermediaryDisplay to signing step

* Move contract info below content on WC screen

* Add tests
@FrederikBolding
Reduce spacing on add account flow (MyCryptoHQ#4242)
e5f5411
@FrederikBolding
Hide membership panel for non-members (MyCryptoHQ#4243)
9a0b2b2 
* Hide membership panel for non-members

* Tweak heights

* Remove topnav item
@FrederikBolding
Fix TX status issue with contract deployment transactions (MyCryptoHQ…
1203aef 
…#4246)

* Fix TX status issue with contract deployment transactions

* Fix type
@FrederikBolding
GIV airdrop detection (MyCryptoHQ#4240)
f97c4ad 
* Add initial code for GIV drop detection

* Add API url and processing for amounts

* Add test

* Add alternative XDAI node

* Fix test
@FrederikBolding
Enable 1559 on xDAI (MyCryptoHQ#4249)
85db86b 
* Enable 1559 for xDAI

* Change trigger for showing fee fiat value slightly
@FrederikBolding
Add contract deployed address to receipt (MyCryptoHQ#4247)
92023df 
* Add contract deployed address to receipt

* Small fixes to contract interaction box

* Revert small change to contract interaction box visibility

* Fix copy

* Add test
@FrederikBolding
Remove membership action (MyCryptoHQ#4255)
bce813c
@FrederikBolding
Add membership construction notice (MyCryptoHQ#4256)
815abd4
@FrederikBolding
Claims performance improvements (MyCryptoHQ#4248)
d850608 
* Persist claims data and determine whether to request new information based on that

* Remove slice from root reducer

* Fix tests

* Add more tests

* Increase timeout a bit

* Fix E2E test
@FrederikBolding
GO chain fixes (MyCryptoHQ#4262)
bb54a93 
* Simplify and fix validateTxFee

* Simplify further

* Increaes GO minimum gas price
@FrederikBolding
Add more 1559 support for more chains (MyCryptoHQ#4264)
0ca2ade 
* Add more 1559 support for more chains

* Enable 1559 support for EWC and Volta
@FrederikBolding
Bump deps (MyCryptoHQ#4266)
c038ae4
@FrederikBolding
Fix send assets data bug (MyCryptoHQ#4265)
e1be017
@FrederikBolding
Remove membership rewards button (MyCryptoHQ#4271)
eab9fa1 
* Remove rewards button

* Fix tests
@Mrtenz
Run prettier in CI workflow (MyCryptoHQ#4276)
33fb39c 
* Run prettier in CI workflow

* Fix inproperly formatted files

* Update lint configs and run format

* Revert matomo.js

* Use matomo.js from master
@FrederikBolding
Update GridPlus DPaths (MyCryptoHQ#4244)
76ef91d 
* Update GridPlus DPaths

* Bump wallets lib and a few other deps

* Run Prettier
@Mrtenz
Update ESLint config to enforce "===" and "??" (MyCryptoHQ#4280)
144b8aa 
* Update ESLint config to enforce "===" and "??"

* Fix lint errors

* Fix one more instance of ||

* Disable rule for one line
@FrederikBolding
Fix an issue with missing base assets (MyCryptoHQ#4284)
e708bf0 
* Fix an issue with missing base assets

* Add test case for bug
@FrederikBolding
Bump wallets lib (MyCryptoHQ#4285)
4001780 
* Bump wallets lib

* Fix Snyk warning for simple-get
@BlocksScanIO
Add Support for XinFin (XDC) Network. (MyCryptoHQ#4250)
1ff2fef 
* Add Support for XinFin ( XDC ) Network.

* Update assets.ts

* Update generateDefaultValues.spec.ts
@FrederikBolding
Use Infura for archival node (MyCryptoHQ#4320)
27e13e8 
* Use Infura for archival node

* Change secret
@FrederikBolding
Fix MetaMask casing (MyCryptoHQ#4317)
fc57b58 
* Fix MetaMask casing

* Fix another issue with MM casing
@FrederikBolding
Add CAD and DKK (MyCryptoHQ#4316)
abf7135
@FrederikBolding
Simplify EIP 1559 support detection (MyCryptoHQ#4119)
d242d63
@FrederikBolding
@mycrypto/[email protected] (MyCryptoHQ#4324)
6559b7e 
* Bump wallets lib

* Update mock
@FrederikBolding
Bump dependencies (MyCryptoHQ#4325)
3b68608
@FrederikBolding
@mycrypto/[email protected] (MyCryptoHQ#4328)
457ec2f
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.