jyapayne/langflow
1
0
Fork 0
Code Issues Pull requests Projects Releases 1 Packages Wiki Activity Actions

fix: fix bug on row selection at IO modal (#2627)

Browse source 
* feat: optimize filtering of messages in SessionView component

The code changes in this commit optimize the filtering of messages in the SessionView component. The useMemo hook is used to memoize the filteredMessages array, which improves performance by avoiding unnecessary re-rendering.

* [autofix.ci] apply automated fixes

---------

Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com>
This commit is contained in:
anovazzi1 2024-07-12 12:28:56 -03:00 • committed by GitHub
commit 229b8680f7
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
1 changed files with 11 additions and 7 deletions
Download patch file Download diff file Expand all files Collapse all files

18
src/frontend/src/modals/IOModal/components/SessionView/index.tsx
View file

@ -7,7 +7,7 @@ import {
SelectionChangedEvent,
} from "ag-grid-community";
import cloneDeep from "lodash/cloneDeep";
import { useState } from "react";
import { useMemo, useState } from "react";
import TableComponent from "../../../../components/tableComponent";
import useRemoveMessages from "../../../../pages/SettingsPage/pages/messagesPage/hooks/use-remove-messages";
import useUpdateMessage from "../../../../pages/SettingsPage/pages/messagesPage/hooks/use-updateMessage";
@ -56,12 +56,16 @@ export default function SessionView({
});
}
let filteredMessages = session
? messages.filter((message) => message.session_id === session)
: messages;
filteredMessages = id
? filteredMessages.filter((message) => message.flow_id === id)
: filteredMessages;
const filteredMessages = useMemo(() => {
let filteredMessages = session
? messages.filter((message) => message.session_id === session)
: messages;
filteredMessages = id
? filteredMessages.filter((message) => message.flow_id === id)
: filteredMessages;
return filteredMessages;
}, [session, id, messages]);
return isFetching > 0 ? (
<div className="flex h-full w-full items-center justify-center align-middle">
<Loading></Loading>