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.

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

Improvement of the app performance #248

Merged
merged 1 commit into from
Apr 10, 2024
Merged

Improvement of the app performance #248

merged 1 commit into from
Apr 10, 2024

Conversation

Damola18
Copy link
Contributor

@Damola18 Damola18 commented Apr 8, 2024

Improvement of the app performance

This pr improves the performance of the app by reducing the LCP from 7.5s to 1.2s

Linked issues

closes EveripediaNetwork/issues#2490

image

Copy link

stackblitz bot commented Apr 8, 2024

Review PR in StackBlitz Codeflow Run & review this pull request in StackBlitz Codeflow.

Copy link

vercel bot commented Apr 8, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
iq-dashboard ✅ Ready (Inspect) Visit Preview Apr 8, 2024 7:28pm

Copy link

codeclimate bot commented Apr 8, 2024

Code Climate has analyzed commit 8456518 and detected 0 issues on this pull request.

View more on Code Climate.

Copy link
Member

@Royal-lobster Royal-lobster left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Damola18 The LCP issue is with braindao.org - https://github.com/EveripediaNetwork/braindao-ui 😅 but we can merge this PR cuz it adds needed alt tags. make sure to detach the issue before merging

@Damola18
Copy link
Contributor Author

Damola18 commented Apr 9, 2024

@Damola18 The LCP issue is with braindao.org - https://github.com/EveripediaNetwork/braindao-ui 😅 but we can merge this PR cuz it adds needed alt tags. make sure to detach the issue before merging

Oh, the image that was attached to the issue got me. I will detach and ensure a new issue is attached to this then.

@Damola18
Copy link
Contributor Author

Damola18 commented Apr 9, 2024

@Damola18 The LCP issue is with braindao.org - https://github.com/EveripediaNetwork/braindao-ui 😅 but we can merge this PR cuz it adds needed alt tags. make sure to detach the issue before merging

Oh, the label that was attached to the issue got me. I will detach and ensure a new issue is attached to this then.

@Damola18 Damola18 self-assigned this Apr 10, 2024
@Damola18 Damola18 merged commit 7fba84d into main Apr 10, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Improve the performance issue on BrainDAO Page
3 participants