forked from jsnbuchanan/crowd-funder-for-time-pwa
Merge pull request 'fix: persist identity names per user instead of globally' (#191) from switching-identities-change-name into master
Reviewed-on: trent_larson/crowd-funder-for-time-pwa#191
This commit is contained in:
@@ -84,7 +84,8 @@ export default class UserNameDialog extends Vue {
|
|||||||
*/
|
*/
|
||||||
async open(aCallback?: (name?: string) => void) {
|
async open(aCallback?: (name?: string) => void) {
|
||||||
this.callback = aCallback || this.callback;
|
this.callback = aCallback || this.callback;
|
||||||
const settings = await this.$settings();
|
// Load from account-specific settings instead of master settings
|
||||||
|
const settings = await this.$accountSettings();
|
||||||
this.givenName = settings.firstName || "";
|
this.givenName = settings.firstName || "";
|
||||||
this.visible = true;
|
this.visible = true;
|
||||||
}
|
}
|
||||||
@@ -95,7 +96,18 @@ export default class UserNameDialog extends Vue {
|
|||||||
*/
|
*/
|
||||||
async onClickSaveChanges() {
|
async onClickSaveChanges() {
|
||||||
try {
|
try {
|
||||||
await this.$updateSettings({ firstName: this.givenName });
|
// Get the current active DID to save to user-specific settings
|
||||||
|
const settings = await this.$accountSettings();
|
||||||
|
const activeDid = settings.activeDid;
|
||||||
|
|
||||||
|
if (activeDid) {
|
||||||
|
// Save to user-specific settings for the current identity
|
||||||
|
await this.$saveUserSettings(activeDid, { firstName: this.givenName });
|
||||||
|
} else {
|
||||||
|
// Fallback to master settings if no active DID
|
||||||
|
await this.$saveSettings({ firstName: this.givenName });
|
||||||
|
}
|
||||||
|
|
||||||
this.visible = false;
|
this.visible = false;
|
||||||
this.callback(this.givenName);
|
this.callback(this.givenName);
|
||||||
} catch (error) {
|
} catch (error) {
|
||||||
|
|||||||
@@ -110,10 +110,22 @@ export default class NewEditAccountView extends Vue {
|
|||||||
* @async
|
* @async
|
||||||
*/
|
*/
|
||||||
async onClickSaveChanges() {
|
async onClickSaveChanges() {
|
||||||
await this.$updateSettings({
|
// Get the current active DID to save to user-specific settings
|
||||||
|
const settings = await this.$accountSettings();
|
||||||
|
const activeDid = settings.activeDid;
|
||||||
|
|
||||||
|
if (activeDid) {
|
||||||
|
// Save to user-specific settings for the current identity
|
||||||
|
await this.$saveUserSettings(activeDid, {
|
||||||
firstName: this.givenName,
|
firstName: this.givenName,
|
||||||
lastName: "", // deprecated, pre v 0.1.3
|
|
||||||
});
|
});
|
||||||
|
} else {
|
||||||
|
// Fallback to master settings if no active DID
|
||||||
|
await this.$saveSettings({
|
||||||
|
firstName: this.givenName,
|
||||||
|
});
|
||||||
|
}
|
||||||
|
|
||||||
this.$router.back();
|
this.$router.back();
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user