fix: fixed identity rename props

PolymeshAssociation · Oct 27, 2023 · 77db160 · 77db160
1 parent b95f0ba
commit 77db160
Show file tree

Hide file tree

Showing 6 changed files with 11 additions and 13 deletions.
diff --git a/packages/core/src/background/handlers/Extension.ts b/packages/core/src/background/handlers/Extension.ts
@@ -143,10 +143,10 @@ export default class Extension extends DotExtension {
   }
 
   private polyIdentityRename({
-    address,
+    did,
     name,
   }: RequestPolyIdentityRename): boolean {
-    renameIdentity(address, name);
+    renameIdentity(did, name);
 
     return true;
   }

diff --git a/packages/core/src/background/types.ts b/packages/core/src/background/types.ts
@@ -71,7 +71,7 @@ export interface RequestPolyCallDetails {
 }
 
 export interface RequestPolyIdentityRename {
-  address: string;
+  did: string;
   name: string;
 }
 

diff --git a/packages/core/src/store/features/identities.ts b/packages/core/src/store/features/identities.ts
@@ -24,7 +24,7 @@ type SetIdentityPayload = {
   };
 };
 type RenameIdentityPayload = {
-  account: string;
+  did: string;
   name: string;
 };
 
@@ -42,10 +42,8 @@ const identitiesSlice = createSlice({
       state.currentDids[account] = currentDid;
     },
     renameIdentity(state, action: PayloadAction<RenameIdentityPayload>) {
-      const { account, name } = action.payload;
-      const currentDid = state.currentDids[account];
-
-      state.dids[currentDid].alias = name;
+      const { did, name } = action.payload;
+      state.dids[did].alias = name;
     },
     removeCurrentIdentity(state, action: PayloadAction<string>) {
       delete state.currentDids[action.payload];

diff --git a/packages/core/src/store/setters.ts b/packages/core/src/store/setters.ts
@@ -21,8 +21,8 @@ function setSelectedAccount(account: string): void {
   store.dispatch(accountActions.selectAccount(account));
 }
 
-function renameIdentity(account: string, name: string) {
-  store.dispatch(identityActions.renameIdentity({ account, name }));
+function renameIdentity(did: string, name: string) {
+  store.dispatch(identityActions.renameIdentity({ did, name }));
 }
 
 function resetState() {

diff --git a/packages/ui/src/Popup/Accounts/AccountsContainer.tsx b/packages/ui/src/Popup/Accounts/AccountsContainer.tsx
@@ -68,7 +68,7 @@ export const AccountsContainer: FC<Props> = ({
       return;
     }
 
-    await renameIdentity(currentAccount?.address, newAlias);
+    await renameIdentity(currentAccount?.did, newAlias);
     stopEditAlias();
     onAction();
   };

diff --git a/packages/ui/src/messaging.ts b/packages/ui/src/messaging.ts
@@ -375,10 +375,10 @@ export async function subscribePolyStatus(
 }
 
 export async function renameIdentity(
-  address: string,
+  did: string,
   name: string
 ): Promise<boolean> {
-  return polyMessage('poly:pri(identity.rename)', { address, name });
+  return polyMessage('poly:pri(identity.rename)', { did, name });
 }
 
 export async function getPolyCallDetails(