add TODOs

This commit is contained in:
Sasha 2023-02-23 13:03:02 +01:00
parent 450418db23
commit 554ef2ace8
No known key found for this signature in database
2 changed files with 6 additions and 4 deletions

View File

@ -50,6 +50,7 @@ const useCreateNode = <N extends Waku, T = {}>(
return () => {
cancelled = true;
};
// TODO: missing any dependencies, it will prevent consecutive update if options change
}, []);
return {

View File

@ -33,12 +33,12 @@ export const useStoreMessages = (
const [messages, setMessage] = React.useState<IDecodedMessage[]>([]);
const pushMessage = React.useCallback(
(messages: IDecodedMessage[]): void => {
if (!messages || !messages.length) {
(newMessages: IDecodedMessage[]): void => {
if (!newMessages || !newMessages.length) {
return;
}
setMessage((prev) => [...prev, ...messages]);
setMessage((prev) => [...prev, ...newMessages]);
},
[setMessage],
);
@ -83,7 +83,8 @@ export const useStoreMessages = (
return () => {
cancelled = true;
};
}, [node, decoder, options, pushMessage, setError, setLoading]);
// TODO: missing dependency on options, it will prevent consecutive update if options change
}, [node, decoder, pushMessage, setError, setLoading]);
return {
error,