LaconicNetwork/vega-frontend-monorepo
3
0
Fork 0
Code Issues Pull Requests 1 Packages Projects Releases Wiki Activity

feat(trading): do not pin row of existing account as it will become duplicated (#2942)

Browse Source
This commit is contained in:
m.ray 2023-02-20 10:59:01 -05:00 committed by GitHub
parent 4286bc37b3
commit c73b01d549
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
libs/accounts/src/lib/accounts-table.spec.tsx
View File

@ -76,7 +76,7 @@ describe('AccountsTable', () => {
expect(rows.length).toBe(6);
});
it('should add first asset as pinned', async () => {
it('should not add first asset as pinned', async () => {
await act(async () => {
render(
<AccountTable
@ -93,7 +93,7 @@ describe('AccountsTable', () => {
);
});
const rows = await screen.findAllByRole('row');
expect(rows.length).toBe(9);
expect(rows.length).toBe(6);
});
it('should get correct account data', () => {

2
libs/accounts/src/lib/accounts-table.tsx
View File

@ -58,7 +58,7 @@ export const AccountTable = forwardRef<AgGridReact, AccountTableProps>(
};
}
}
return currentPinnedAssetRow;
return undefined;
}, [pinnedAssetId, props.pinnedAsset, props.rowData]);
return (