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

[APP-1049]: (feat): Backups V2 #5310

Merged
merged 106 commits into from
Mar 27, 2024
Merged

[APP-1049]: (feat): Backups V2 #5310

merged 106 commits into from
Mar 27, 2024

Conversation

walmat
Copy link
Contributor

@walmat walmat commented Jan 3, 2024

Fixes APP-1049

What changed (plus any additional context for devs)

Screen recordings / screenshots

What to test

* (feat): add backups global config

* stash

* add migration for setting initial backup method

* add method to migrations array

* revert pbx change

* change backup status value
Copy link

linear bot commented Jan 3, 2024

walmat and others added 22 commits January 4, 2024 10:13
… and `backupStatus` (#5313)

* [APP-1052]: (feat): adjust settings page row

* fix migration and hook for useWallets

* simplify an if statement
* wallets and backups screen progress

* manual backups progress

* progress

* feat(backups): more work on wallets and backups

* revert pbx

* rev init wallet change

* final touches on wallets and backup view
* save progress

* cleanup and finish secret phrase / private key viewiing screens

---------

Co-authored-by: Matthew Wall <matthew.wallt@gmail.com>
* progress on cloud backup flow

* cloud backup flow progress
* progress on restore from cloud

* progress on choose backup flow

* f

* test signed commit

* backups v2 restore from icloud flow done

* progress on restore step
@walmat walmat marked this pull request as ready for review February 5, 2024 22:34
Copy link
Contributor

@ibrahimtaveras00 ibrahimtaveras00 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All issues have been resolved, QA Passed ✅

Copy link
Member

@brunobar79 brunobar79 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good stuff! Added minor comments but LGTM!

src/components/backup/BackupCloudStep.tsx Outdated Show resolved Hide resolved
if (privateKeyAddress) {
const privateKeyData = await loadPrivateKey(privateKeyAddress, false);
if (privateKeyData === -1 || privateKeyData === -2 || !privateKeyData) {
setSectionState(SecretDisplayStates.noSeed);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this might be one of the places were we need to add the alert to contact support, just because this needs to be backwards compatible cc: @jin

src/hooks/useManageCloudBackups.ts Outdated Show resolved Hide resolved
},
async (buttonIndex: number) => {
if (buttonIndex === 0) {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

are we removing the option to restore and older backup?

src/keychain/index.ts Outdated Show resolved Hide resolved
src/keychain/index.ts Outdated Show resolved Hide resolved
src/keychain/index.ts Outdated Show resolved Hide resolved
src/model/backup.ts Outdated Show resolved Hide resolved
src/model/backup.ts Outdated Show resolved Hide resolved
const valueStr = backedUpData[key];
const parsedValue = JSON.parse(valueStr);

let secretPhraseOrOldAndroidBackupPrivateKey: string | any; // TODO: Strengthen this type
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We have a type for pkeys

Co-authored-by: Bruno Barbieri <1247834+brunobar79@users.noreply.github.com>
Copy link
Contributor

@skylarbarrera skylarbarrera left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🫡

src/handlers/walletReadyEvents.ts Show resolved Hide resolved
walmat and others added 4 commits March 27, 2024 10:39
Co-authored-by: Bruno Barbieri <1247834+brunobar79@users.noreply.github.com>
@skylarbarrera skylarbarrera merged commit 783f120 into develop Mar 27, 2024
5 of 6 checks passed
@skylarbarrera skylarbarrera deleted the @matthew/APP-1049 branch March 27, 2024 18:01
BrodyHughes added a commit that referenced this pull request Apr 2, 2024
* 'develop' of github.com:rainbow-me/rainbow:
  Only hold the active tab ref in BrowserContext (#5579)
  Dapp browser: disable tab closing for empty state (#5573)
  Browser: fix ref assignment, back/forward navigation (#5578)
  Browser: fully eliminate reloading issues (#5576)
  browser: static trending dapps (#5561)
  bump swaps sdk (#5574)
  fix gitignore (#5571)
  bump (#5570)
  Fix browser context menu not updating (#5569)
  ⚡️ Fast browser (#5566)
  [APP-1049]: (feat): Backups V2 (#5310)
  fix: search by contract address (#5563)
This was referenced Apr 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backups v2 release for release blockers and release candidate branches
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants