Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Make all the code base have memoization and improve ESLint rules to enforce it #37

Closed
devkiloton opened this issue Dec 22, 2023 · 0 comments · Fixed by #53
Closed

Make all the code base have memoization and improve ESLint rules to enforce it #37

devkiloton opened this issue Dec 22, 2023 · 0 comments · Fixed by #53
Assignees
@devkiloton
Labels
Effort: ⭐️⭐️⭐️ Oh boy Priority: ⭐️⭐️⭐️ It'll be crucial in the next few days/week

Comments

@devkiloton
Copy link
Owner

  • Functions must to be wrapped with useCallback()
  • Components must to be wrapped with React.memo()
@devkiloton devkiloton self-assigned this Dec 22, 2023
@devkiloton devkiloton mentioned this issue Jan 6, 2024
Open
@devkiloton devkiloton added Priority: ⭐️⭐️⭐️ It'll be crucial in the next few days/week and removed Priority: ⭐️⭐️ It must be implemented ASAP labels Jan 14, 2024
@devkiloton devkiloton mentioned this issue Jan 14, 2024
Merged
devkiloton added a commit that referenced this issue Jan 14, 2024
@devkiloton
Merge pull request #53 from devkiloton/feature/37
4d9fc2b 
Make all the code base have memoization and improve ESLint rules to enforce it #37
@devkiloton devkiloton linked a pull request Jan 14, 2024 that will close this issue
Make all the code base have memoization and improve ESLint rules to enforce it #37 #53
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@devkiloton devkiloton

Labels
Effort: ⭐️⭐️⭐️ Oh boy Priority: ⭐️⭐️⭐️ It'll be crucial in the next few days/week
Projects
Invetu's backlog
Status: Done
Milestone
No milestone
1 participant
@devkiloton