Differential D14936

[lib] extract getting store and database version to a separate function
ClosedPublic
Actions

Authored by kamil on Jul 16 2025, 1:53 PM.

Details

Reviewers

bartek

Commits

rCOMMe6fd90c29800: [lib] extract getting store and database version to a separate function

Summary

ENG-10812.

We need to store results in case of error, so when checking database versions fails, we want to store the version too, because of that, we want to fetch both at once.

Depends on D14935

Test Plan

Flow

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

kamil created this revision.Jul 16 2025, 1:53 PM

kamil held this revision as a draft.

Herald added subscribers: tomek, ashoat. · View Herald TranscriptJul 16 2025, 1:53 PM

kamil edited the summary of this revision. (Show Details)Jul 16 2025, 1:54 PM

kamil added a child revision: D14937: [lib] implement `BackupIsNewerError` class.Jul 16 2025, 1:56 PM

Harbormaster completed remote builds in B34574: Diff 48746.Jul 16 2025, 2:12 PM

kamil published this revision for review.Jul 16 2025, 2:34 PM

bartek accepted this revision.Jul 16 2025, 2:53 PM

This revision is now accepted and ready to land.Jul 16 2025, 2:53 PM

Closed by commit rCOMMe6fd90c29800: [lib] extract getting store and database version to a separate function. · Explain WhyJul 17 2025, 9:33 AM

This revision was automatically updated to reflect the committed changes.

kamil added a commit: rCOMMe6fd90c29800: [lib] extract getting store and database version to a separate function.

Revision Contents
Changeset List

Path

Size

lib/

backup/

use-user-data-restore.js

92 lines

Diff 48755

View Options

lib/backup/use-user-data-restore.js

[lib] extract getting store and database version to a separate functionClosedPublicActions