Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Feature/intervention #2343
base: develop
Are you sure you want to change the base?
Feature/intervention #2343
Changes from 43 commits
23261b4
140cb0d
a3e6a30
067d46d
3afbe9e
3e17a09
d861be7
b4f31fc
411c121
2d6de25
8b9ccf9
1659928
282b8e4
96a027b
43d3856
9f9021c
8bfe467
7829842
b3cfe1b
9c4e9b5
bcabeb4
578a4c2
9a88fe8
2d82886
57d4527
59b6165
0b86b1f
588bfc2
171adf8
b22fc3a
a7555c3
4c6de8d
1ac22f6
6ea3699
7c1e9e9
6bf4248
548994b
555e3ec
d7dc806
36de55d
72797b1
98c4379
0225fc9
6aa75f4
b84ea21
364dae1
b80d951
78492b4
9f802c2
c1d748f
10096f6
b51cf14
a35837b
01c9cfd
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🛠️ Refactor suggestion
Move
createCardData
to a dedicated utility file.For better maintainability and reusability, move this function to a separate utility file. This aligns with the single responsibility principle and makes the code more modular.
Create a new file
src/features/projectsV2/ProjectDetails/utils/metadata.ts
:There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🛠️ Refactor suggestion
Optimize useMemo implementation.
📝 Committable suggestion
🧰 Tools
🪛 Biome (1.9.4)
[error] 118-119: Change to an optional chain.
Unsafe fix: Change to an optional chain.
(lint/complexity/useOptionalChain)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🛠️ Refactor suggestion
Simplify sampleInterventionSpeciesImages logic.
The current implementation has unnecessary complexity and potential undefined access.
📝 Committable suggestion
🧰 Tools
🪛 Biome (1.9.4)
[error] 132-139: Change to an optional chain.
Unsafe fix: Change to an optional chain.
(lint/complexity/useOptionalChain)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fix duplicate keys and improve conditional rendering in content array.
There are several issues in the content array that need to be addressed:
📝 Committable suggestion
🧰 Tools
🪛 Biome (1.9.4)
[error] 192-193: Change to an optional chain.
Unsafe fix: Change to an optional chain.
(lint/complexity/useOptionalChain)