Skip to content

Conversation

@AresPhoenix345
Copy link

Description

  • Added "Academy" navigation item beside Cloud and Kanvas
  • Linked to the official Academy page (https://layer5.io/learn/learning-paths)
  • Used consistent styling with other project icons
  • Opens in new tab for external link
  • Fixed the alt tag in images to reflect original image's alt

Notes for Reviewers

This PR fixes #843

Signed commits

  • Yes, I signed my commits.

Linked to the official Academy page
Used consistent styling with other project icons
Opens in new tab for external link

Signed-off-by: phoenixcodeforces-blip <[email protected]>
@netlify
Copy link

netlify bot commented Oct 23, 2025

Deploy Preview for bejewelled-pegasus-b0ce81 ready!

Name Link
🔨 Latest commit 311fbfb
🔍 Latest deploy log https://app.netlify.com/projects/bejewelled-pegasus-b0ce81/deploys/68fa050cf676590009ef5b57
😎 Deploy Preview https://deploy-preview-844--bejewelled-pegasus-b0ce81.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@kishore08-07
Copy link
Contributor

Hey @phoenixcodeforces-blip,
I was also working on this issue. Just a quick note , it’s best to get the issue assigned to you first or sync up with the person already handling it. That way, we can avoid overlap and keep things running smoothly. Also don’t forget to go through the contributors guidelines , that will help you get a better idea of how we collaborate here.

@AresPhoenix345
Copy link
Author

Hey @phoenixcodeforces-blip, I was also working on this issue. Just a quick note , it’s best to get the issue assigned to you first or sync up with the person already handling it. That way, we can avoid overlap and keep things running smoothly. Also don’t forget to go through the contributors guidelines , that will help you get a better idea of how we collaborate here.

@kishore08-07 thanks for reaching out, you may copy the code from my diff and I will close this PR.
Just to remind you there is a tag in repositories, those who had a merged PR get contributor, the maintainers are usually given member tag. yours did not had any. So I did not tag/ask for permission for implementing it. Since you could not assign it to me.
No hard feelings, but the implementation was quick so I did not delay and Raised the PR.

@kishore08-07
Copy link
Contributor

@phoenixcodeforces-blip
All good! I think there was a slight misunderstanding with the issue description, but I really appreciate your initiative in volunteering and keeping an eye on issues across Layer5 projects. It’s fantastic to see your enthusiasm and contributions.
Happy contributing! 🤝

@Rajesh-Nagarajan-11
Copy link
Member

@phoenixcodeforces-blip Since this PR doesn’t address the actual issue, I’ll go ahead and close it. Thanks for showing interest , we have plenty of other issues you can check out!

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add Academy to Project Navigator

3 participants