Skip to content

Commit

Permalink
Fix: remove failure to import legacy account notification.
Browse files Browse the repository at this point in the history
- It was being displayed even on a fresh install.
- We are already displaying a notification when there are accounts to be
imported and we ask the user on what to do.

Signed-off-by: Camila Ayres <[email protected]>
  • Loading branch information
camilasan committed Jan 24, 2024
1 parent 81847f9 commit 0b1bbdd
Showing 1 changed file with 1 addition and 3 deletions.
4 changes: 1 addition & 3 deletions src/gui/accountmanager.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -178,6 +178,7 @@ bool AccountManager::restoreFromLegacySettings()
legacyCfgFileParentFolder + legacyCfgFileNamePath,
legacyCfgFileGrandParentFolder + legacyCfgFileRelativePath};

auto displayMessageBoxWarning = false;
for (const auto &configFile : legacyLocations) {
auto oCSettings = std::make_unique<QSettings>(configFile, QSettings::IniFormat);
if (oCSettings->status() != QSettings::Status::NoError) {
Expand Down Expand Up @@ -263,9 +264,6 @@ bool AccountManager::restoreFromLegacySettings()
return true;
}

QMessageBox::information(nullptr,
tr("Legacy import"),
tr("Could not import accounts from legacy client configuration."));
return false;
}

Expand Down

0 comments on commit 0b1bbdd

Please sign in to comment.