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

Merge migrations to a single tool #4218

Merged
merged 12 commits into from
Dec 3, 2021
Merged

Conversation

FrederikBolding
Copy link
Collaborator

No description provided.

@mycrypto-bot
Copy link
Collaborator

Staging build: MyCryptoBuilds

@FrederikBolding
Copy link
Collaborator Author

[sc-9143]

@shortcut-integration
Copy link

This pull request has been linked to Shortcut Story #9143: Tools: Merge migrations.

@codecov
Copy link

codecov bot commented Nov 30, 2021

Codecov Report

Merging #4218 (7822070) into master (e9b5c88) will increase coverage by 0.14%.
The diff coverage is 97.50%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #4218      +/-   ##
==========================================
+ Coverage   78.49%   78.63%   +0.14%     
==========================================
  Files         574      572       -2     
  Lines       12402    12420      +18     
  Branches     3064     3069       +5     
==========================================
+ Hits         9735     9767      +32     
+ Misses       2634     2620      -14     
  Partials       33       33              
Impacted Files Coverage Δ
src/components/ActionsPanel/constants.ts 85.71% <ø> (ø)
src/components/Icon/Icon.tsx 95.74% <ø> (ø)
src/components/TransactionFlow/helpers.ts 86.66% <ø> (ø)
src/config/features.ts 100.00% <ø> (ø)
src/config/routePaths.ts 100.00% <ø> (ø)
src/features/Layout/Navigation/constants.ts 100.00% <ø> (ø)
...atures/TokenMigration/AaveTokenMigration/config.ts 100.00% <ø> (ø)
...tures/TokenMigration/AaveTokenMigration/helpers.ts 100.00% <ø> (ø)
...eatures/TokenMigration/AntTokenMigration/config.ts 100.00% <ø> (ø)
...atures/TokenMigration/AntTokenMigration/helpers.ts 100.00% <ø> (ø)
... and 15 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e9b5c88...7822070. Read the comment docs.

@FrederikBolding FrederikBolding temporarily deployed to STAGING December 1, 2021 11:34 Inactive
@FrederikBolding FrederikBolding temporarily deployed to STAGING December 1, 2021 12:31 Inactive
@FrederikBolding FrederikBolding marked this pull request as ready for review December 1, 2021 14:15
src/components/TextSelector.tsx Outdated Show resolved Hide resolved
@FrederikBolding FrederikBolding temporarily deployed to STAGING December 2, 2021 14:47 Inactive
@FrederikBolding FrederikBolding temporarily deployed to STAGING December 3, 2021 12:39 Inactive
@FrederikBolding FrederikBolding merged commit c50dc20 into master Dec 3, 2021
@FrederikBolding FrederikBolding deleted the fb/migrations-merge branch December 3, 2021 13:39
GuillaumeRx pushed a commit that referenced this pull request Feb 3, 2022
* Move files around

* Move more files

* Add migration selector

* Remove unused routes

* Update nav icon

* Add test

* Fix TextSelector padding

* Update copy slightly

* Fix test

* Use default param value

* Auto select a default migration if possible

* Add migration E2E
GuillaumeRx pushed a commit that referenced this pull request Feb 3, 2022
* Move files around

* Move more files

* Add migration selector

* Remove unused routes

* Update nav icon

* Add test

* Fix TextSelector padding

* Update copy slightly

* Fix test

* Use default param value

* Auto select a default migration if possible

* Add migration E2E
GuillaumeRx pushed a commit that referenced this pull request Feb 3, 2022
* Move files around

* Move more files

* Add migration selector

* Remove unused routes

* Update nav icon

* Add test

* Fix TextSelector padding

* Update copy slightly

* Fix test

* Use default param value

* Auto select a default migration if possible

* Add migration E2E
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.

3 participants