(Fix): chart review component optimized

[Madhu-mac]

Requirements

• This PR has a title that briefly describes the work done including the ticket number. If there is a ticket, make sure your PR title includes a conventional commit label. See existing PR titles for inspiration.
• My work conforms to the OpenMRS 3.0 Styleguide and design documentation.
• My work includes tests or is validated by existing tests.

Summary

This PR will add functionality in the code by adding usememo for better callback usage & also added error handling on top of it for better optimized performance and readability in the code

Screenshots

Related Issue

Other

[Madhu-mac]

@jwnasambu @denniskigen
Can you verify this PR once

[jwnasambu]

@Madhu-mac Kindly attach the ticket link of the issue you are working on to the Related Issue section for easy review.

[Madhu-mac]

@jwnasambu It doesn't have any tickets raised on this. I've added enhancement to the code for optimization and better results. I've also tested this locally

[brandones]

Hi @Madhu-mac it's not obvious to me that this would improve performance. Whether useMemo improves performance depends on what it's being used for. Could you please provide some evidence that this improves performance?

[denniskigen]

I agree with @brandones here. It would help to have at the very least some before and after profiling data from the React Dev tools to support the claim. Also, it'd be ideal to file a ticket first related to addressing performance concerns in the Patient Chart. Feel free to reopen this PR when you have these pre-requisites taken care of. Thanks!