search

KaotoIO / kaoto

Next version of the UI of the Kaoto project
Apache License 2.0
22 stars 21 forks source link

fix(991): Removing useMemo() from the code #1168

Closed shivamG640 closed 1 week ago

shivamG640 commented 1 week ago

Description

This PR is related to a previous PR, where I added the following two changes:

  1. Removed the useMemo() calls as I found out that it is unnecessarily used and the underlying code is getting executed all the time the component is re-executed as we inject an object as it dependency. I might be incorrect on this so please let me know if we need a discussion over this.
  2. Modified the id to make it unique for all the groups in the expandable section.
shivamG640 commented 1 week ago

@lordrip , Please have a look at this!