Skip to content

Improved Event page Performance #6988

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
Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Improved Event page Performance #6988

wants to merge 1 commit into from

Conversation

@Vaibhavsg17
Copy link

Description
This PR updates the cache headers for static assets and HTML files to improve page performance scores, especially for Mobile, as identified in PageSpeed Insights.

This PR fixes #6929

Notes for Reviewers
This PR is raised primarily to enable testing via the preview deployment link. Once the testing is complete and results are verified, I will notify the team so that it can be merged.

Signed commits

  • Yes, I signed my commits.

@Vaibhavsg17
Copy link
Author

Hi @Rajesh-Nagarajan-11,

Could you please guide me on how to obtain the preview deployment link for this PR? I’d like to test the changes before finalizing the merge.

Thanks in advance!

@Rajesh-Nagarajan-11
Copy link
Member

Kindly sign off your commits please

To add your Signed-off-by line to every commit in this branch:

  1. Ensure you have a local copy of your branch by checking out the pull request locally via command line.
  2. In your local branch, run: git rebase HEAD~2 --signoff
  3. Force push your changes to overwrite the branch: git push --force-with-lease origin improved-event-page-performance

@Rajesh-Nagarajan-11
Copy link
Member

Preview you will get via comment soon

@l5io
Copy link
Contributor

l5io commented Oct 5, 2025

🚀 Preview for commit 88f8929 at: https://68e241c8a82e51e1d56508fb--layer5.netlify.app

Rajesh-Nagarajan-11
Rajesh-Nagarajan-11 requested changes Oct 5, 2025
View reviewed changes
Copy link
Member

@Rajesh-Nagarajan-11 Rajesh-Nagarajan-11 left a comment

Choose a reason for hiding this comment

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

Your changes have reduced the performance to 40%. I think it needs further improvements.

@Vaibhavsg17
Copy link
Author

Vaibhavsg17 commented Oct 5, 2025 •
edited
Loading

image is the same link you used for the PageSpeed Insights ?

(by the way, i am still working on it for more improvements.)

@Rajesh-Nagarajan-11
Copy link
Member

Be specific with url /community/events

@l5io
Copy link
Contributor

l5io commented Oct 5, 2025

🚀 Preview for commit eaf0b4c at: https://68e29a64e3edf2d30071c616--layer5.netlify.app

@leecalcote leecalcote added the hacktoberfest Happy contributing! label Oct 5, 2025
@Namanv0509
Copy link
Member

Thank you for your contribution!
Let's discuss this during the website call today at 5:30 PM IST | 7 AM CT

Add it as an agenda item to the meeting minutes, if you would :)

@Namanv0509 Namanv0509 added the hacktoberfest-accepted Happy contributing! label Oct 7, 2025
@Namanv0509
Copy link
Member

image An improvement 🎉 , can you pull it up to 80% ? @Vaibhavsg17

@Namanv0509
Copy link
Member

image image

This can be helpful for you to work on

@Vaibhavsg17
Copy link
Author

Yes, due to the office, I didn't get enough time.

I'll try my best to pull it till 80%

@Namanv0509
Copy link
Member

@Vaibhavsg17 🙌 Thanks , also please look over the DCO error , make sure you sign off your commits : https://github.com/layer5io/layer5/pull/6988/checks?check_run_id=52301632142

@l5io
Copy link
Contributor

l5io commented Oct 8, 2025

🚀 Preview for commit d8b275e at: https://68e6ed42c157e07a112f7c38--layer5.netlify.app

@l5io
Copy link
Contributor

l5io commented Oct 9, 2025

🚀 Preview for commit ab22607 at: https://68e7e0402559d197582fb6c6--layer5.netlify.app

@Namanv0509
Copy link
Member

@Vaibhavsg17 any update on this PR ?

@Namanv0509 Namanv0509 added the issue/dco Commit sign-off instructions label Oct 10, 2025
@github-actions
Copy link

🚨 Alert! Git Police! We couldn’t help but notice that one or more of your commits is missing a sign-off. A what? A commit sign-off (your email address).

To amend the commits in this PR with your signoff using the instructions provided in the DCO check.

To configure your dev environment to automatically signoff on your commits in the future, see these instructions.

        Be sure to join the community, if you haven't yet and please leave a ⭐ star on the project 😄

1 similar comment
@github-actions
Copy link

🚨 Alert! Git Police! We couldn’t help but notice that one or more of your commits is missing a sign-off. A what? A commit sign-off (your email address).

To amend the commits in this PR with your signoff using the instructions provided in the DCO check.

To configure your dev environment to automatically signoff on your commits in the future, see these instructions.

        Be sure to join the community, if you haven't yet and please leave a ⭐ star on the project 😄

@l5io
Copy link
Contributor

l5io commented Oct 10, 2025

🚀 Preview for commit 562d30c at: https://68e9109cda152711da965635--layer5.netlify.app

@l5io
Copy link
Contributor

l5io commented Oct 13, 2025

🚀 Preview for commit 041cdae at: https://68ec9ad678d7ce6156777246--layer5.netlify.app

@Namanv0509
Copy link
Member

Thank you for your contribution!
Let's discuss this during the website call today at 5:30 PM IST | 7 AM CT

Add it as an agenda item to the meeting minutes, if you would :)

@l5io
Copy link
Contributor

l5io commented Oct 13, 2025

🚀 Preview for commit 968b3e0 at: https://68ecfbac5118d4835587d7c7--layer5.netlify.app

@Rajesh-Nagarajan-11
Copy link
Member

Any updates ? @Vaibhavsg17

@l5io
Copy link
Contributor

l5io commented Oct 18, 2025

🚀 Preview for commit a6a4dc0 at: https://68f3cdaf53e83c2171f816e8--layer5.netlify.app

@Vaibhavsg17
Copy link
Author

yes today i will make final commit with dco also

@Vaibhavsg17
improved performance
2507764 
Signed-off-by: Vaibhavsg17 <vaibhav.s221064102@vcet.edu.in>
@Vaibhavsg17 Vaibhavsg17 force-pushed the improved-event-page-performance branch from a6a4dc0 to 2507764 Compare October 20, 2025 06:40
@github-actions github-actions bot added area/news A noteworthy article, event, happening area/blog New posts or new blog functionality area/learn Related to /learn section area/events area/careers area/resources project/kanvas labels Oct 20, 2025
@l5io
Copy link
Contributor

l5io commented Oct 20, 2025

🚀 Preview for commit 2507764 at: https://68f5dc962e8a871e8bd5e660--layer5.netlify.app

@Rajesh-Nagarajan-11
Copy link
Member

image its getting worst

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Rajesh-Nagarajan-11 Rajesh-Nagarajan-11 Rajesh-Nagarajan-11 requested changes

Assignees

No one assigned

Labels

area/blog New posts or new blog functionality area/careers area/events area/learn Related to /learn section area/news A noteworthy article, event, happening area/resources hacktoberfest Happy contributing! hacktoberfest-accepted Happy contributing! issue/dco Commit sign-off instructions project/kanvas

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[ Performance ] Improve Event page Performance

5 participants

@Vaibhavsg17 @Rajesh-Nagarajan-11 @l5io @Namanv0509 @leecalcote