status-im
/
status-desktop
mirror of https://github.com/status-im/status-desktop.git
2
0
Fork 0
Code Issues Projects Releases Wiki Activity

feat: add error handling on account change

This commit is contained in:
Jonathan Rainville 2020-06-10 14:40:27 -04:00 committed by Iuri Matias
parent be8188078b
commit 65efdf4c71
1 changed files with 19 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

24
ui/app/AppLayouts/Wallet/AccountSettingsModal.qml
View File

@ -1,6 +1,7 @@
import QtQuick 2.12
import QtQuick.Controls 2.3
import QtQuick.Layouts 1.3
import QtQuick.Dialogs 1.3
import "../../../imports"
import "../../../shared"
@ -112,13 +113,26 @@ ModalPopup {
disabled: accountNameInput.text === ""
MessageDialog {
id: changeError
title: "Changing settings failed"
icon: StandardIcon.Critical
standardButtons: StandardButton.Ok
}
onClicked : {
// TODO add message to show validation errors
if (accountNameInput.text === "") return;
console.log('SAVE')
if (accountNameInput.text === "") {
changeError.text = qsTr("Account name cannot be empty")
changeError.open()
}
const error = walletModel.changeAccountSettings(currentAccount.address, accountNameInput.text, selectedColor);
console.log('Error?', error)
// TODO manage errors adding account
if (error) {
changeError.text = error
changeError.open()
return
}
popup.close();
}
}