refactor: reduce the number of renders in Dashboad page #3039
Conversation
Manush-2005
requested review from
derberg,
akshatnema,
magicmatatjahu,
anshgoyalevil,
Mayaleeeee and
asyncapi-bot-eve
as code owners
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
There was a problem hiding this comment.
Welcome to AsyncAPI. Thanks a lot for creating your first pull request. Please check out our contributors guide useful for opening a pull request.
Keep in mind there are also other channels you can use to interact with AsyncAPI community. For more details check out this issue.
✅ Deploy Preview for asyncapi-website ready!Built without sensitive environment variables
To edit notification comments on pull requests, go to your Netlify site configuration. |
asyncapi-bot
requested review from
sambhavgupta0705,
BhaswatiRoy,
J0SAL,
TRohit20 and
VaishnaviNandakumar
|
⚡️ Lighthouse report for the changes in this PR:
Lighthouse ran on https://deploy-preview-3039--asyncapi-website.netlify.app/ |
| @@ -32,6 +32,8 @@ export function filterIssues(issues: Issue[], filters: FiltersType): Issue[] { | |||
| result = result.filter((issue) => issue.area === filters.selectedArea); | |||
| } | |||
|
|
|||
| console.log(result); | |||
There was a problem hiding this comment.
Remove this console log statement.
|
@akshatnema I have done the changes.Please review it. |
akshatnema
changed the title
|
|
||
|
|
There was a problem hiding this comment.
Remove unnecessary spaces.
|
|
||
|
|
|
Ok removed the unnesscary spaces as you requested. |
|
@akshatnema Do I need to make changes in this PR or is it OK ? |
|
/update |
|
/update |
Description
useMemo in Dashboard component to reduce unnecessary renders of repo when page loads.
Also removed scroll errors in Good Frist Issues section.
In the website :
Change:
