perf(kanvas): lazy-load non-critical sections and reduce unused JavaS… #7146
Conversation
|
🚀 Preview for commit 96d1cc0 at: https://6922e1d23100b6619275b4a6--layer5.netlify.app |
|
@Azizah2023 Please remove all unnecessary lint changes such as extra lines and spaces, and keep only the original logic. |
|
🚀 Preview for commit dbaf1ba at: https://6922f3322f7dd379d9da1bc4--layer5.netlify.app |
saurabhraghuvanshii
requested review from
ChetanFTW,
Rajesh-Nagarajan-11 and
kishore08-07
|
🚀 Preview for commit c0877e6 at: https://692591ed3e676a76d57e9e24--layer5.netlify.app |
|
Checks are currently failing, possibly due to the latest Gatsby update |
…cript Signed-off-by: Azizah2023 <A.alshehri@thegarage.sa>
Signed-off-by: Azizah2023 <A.alshehri@thegarage.sa>
Azizah2023
force-pushed
the
perf/kanvas-lazy-load
branch
from
e348de9 to
014680e
Compare
github-actions
bot
added
area/landscape
Signed-off-by: Azizah <121037475+Azizah2023@users.noreply.github.com>
|
@Azizah2023 |
Signed-off-by: Azizah2023 <A.alshehri@thegarage.sa>
Signed-off-by: Azizah <121037475+Azizah2023@users.noreply.github.com>
There was a problem hiding this comment.
@Azizah2023
Please ensure that your commit messages are meaningful rather than repeating the same message, as we need a clear history of changes. Finally, I noticed some files only contain formatting updates (like spacing). Please revert those specific modifications so we can keep the pull request focused strictly on functional code changes.
|
@Azizah2023 Thank you for your contribution! Let's discuss this during the website call today at 6:30 PM IST | 8 AM CT Add it as an agenda item to the meeting minutes, if you would 🙂 |
|
@Azizah2023 Thank you for your contribution! Let's discuss this during the website call tomorrow at 6:30 PM IST | 8 AM CT Add it as an agenda item to the meeting minutes, if you would 🙂 |
Azizah2023
force-pushed
the
perf/kanvas-lazy-load
branch
from
7a93a83 to
a5ed310
Compare
|
🚀 Preview for commit a5ed310 at: https://6936a373e62416d0c58ccb6b--layer5.netlify.app |
|
🚀 Preview for commit 0e6d3de at: https://6936a9d70775ef02ce4727d2--layer5.netlify.app |
|
🚨 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.
|
1 similar comment
|
🚨 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.
|
Signed-off-by: Azizah2023 <A.alshehri@thegarage.sa>
Azizah2023
force-pushed
the
perf/kanvas-lazy-load
branch
from
0e6d3de to
8d2e5ee
Compare
|
🚀 Preview for commit 8d2e5ee at: https://6936d8527994ab06e3a9817b--layer5.netlify.app |
|
🚀 Preview for commit 93b6fde at: https://6937065250b01e7f4f98ee18--layer5.netlify.app |
8 participants
Description
Changes
import()+useEffect.Notes
fix/seo-alt-links-kanvas, which became noisy after syncing with master.This PR fixes #
#7110
Notes for Reviewers
Signed commits