Skip to content

Conversation

vr-varad
Copy link
Contributor

@vr-varad vr-varad commented Jul 4, 2025

Description

This PR fixes #6604

Notes for Reviewers

Signed commits

  • Yes, I signed my commits.

Signed-off-by: Varad Gupta <[email protected]>
@l5io
Copy link
Contributor

l5io commented Jul 4, 2025

🚀 Preview for commit 73b1f1c at: https://6867d361e7e81a86e5933b9a--layer5.netlify.app

Signed-off-by: Varad Gupta <[email protected]>
@vr-varad vr-varad requested a review from zihanKuang July 4, 2025 13:23
@vr-varad vr-varad changed the title Fix: Update Images Fix: Update Outdated Images on Learning Path. Jul 4, 2025
@l5io
Copy link
Contributor

l5io commented Jul 4, 2025

🚀 Preview for commit 503ff4a at: https://6867d886ef0287b4b15f35be--layer5.netlify.app

@vr-varad vr-varad requested a review from FaheemOnHub July 4, 2025 14:00
Copy link
Contributor

@zihanKuang zihanKuang left a comment

Choose a reason for hiding this comment

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

Hi @vr-varad,
The pictures are very fitting! However, regarding the two points below:
For the first one, concerning numbers 2 & 3, I couldn't find number 1.
The second picture is also outdated and could be replaced with the latest version.
Thanks!👍
image
image

Signed-off-by: vr-varad <[email protected]>
@vr-varad
Copy link
Contributor Author

vr-varad commented Jul 7, 2025

Thanks @zihanKuang for reviewing!!
Actually, there is another image for step 1
image
and updated the second pic.

@vr-varad vr-varad requested a review from zihanKuang July 7, 2025 07:09
@l5io
Copy link
Contributor

l5io commented Jul 7, 2025

🚀 Preview for commit d34adf2 at: https://686b758107d2f3e594882fea--layer5.netlify.app

Signed-off-by: vr-varad <[email protected]>
@l5io
Copy link
Contributor

l5io commented Jul 7, 2025

🚀 Preview for commit b7bb070 at: https://686b7aa8deea218408beaf74--layer5.netlify.app

Copy link
Contributor

@zihanKuang zihanKuang left a comment

Choose a reason for hiding this comment

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

LGTM

@zihanKuang zihanKuang merged commit d5fabe7 into layer5io:master Jul 7, 2025
5 of 6 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.

[Learn] Update Outdated Images
3 participants