Skip to content

Conversation

yan00126
Copy link
Contributor

@yan00126 yan00126 commented Jun 5, 2025

Hi @TamimiGitHub / @Mrc0113 / @rjriel,
I updated community links on tutorials site, if you can please review and merge.

thanks

✨ PR Description

Purpose: Update community URLs from "solace.community" to "community.solace.com" across website components and add Netlify optimization plugins.

Main changes:

  • Add image optimization plugin (netlify-plugin-image-optim) and Lighthouse audit plugin to netlify.toml
  • Update community links in header.js, footer.js, and resources.js from "solace.community" to "community.solace.com"
  • Fix incorrect feedback link path in Java hello-world tutorial page markdown

Generated by LinearB AI and added by gitStream.
AI-generated content may contain inaccuracies. Please verify before using. We'd love your feedback! 🚀

@yan00126 yan00126 requested a review from TamimiGitHub June 5, 2025 18:46
@Mrc0113 Mrc0113 self-requested a review June 5, 2025 19:00
@Mrc0113
Copy link
Contributor

Mrc0113 commented Jun 5, 2025

Thanks @yan00126 , looks like we have some build issues we'll have to resolve before we can merge it!

@Mrc0113 Mrc0113 requested review from blainehussey and rjriel June 5, 2025 19:01
Copy link

@gitstream-cm gitstream-cm bot left a comment

Choose a reason for hiding this comment

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

✨ PR Review

The PR updates community links across the site from the old domain to a new one, and includes some configuration cleanup. The changes are straightforward URL updates with one bug fix for an incorrect feedback link.

1 issues detected:

🐞 Bug - Feedback link was pointing to wrong tutorial file

Details: The feedback link was pointing to the wrong tutorial file (publish-subscribe.md instead of hello-world.md). This has been corrected to point to the proper hello-world.md file.
File: src/pages/tutorials/java/hello-world.md

Generated by LinearB AI and added by gitStream.
AI-generated content may contain inaccuracies. Please verify before using. We'd love your feedback! 🚀

Copy link

@gitstream-cm gitstream-cm bot left a comment

Choose a reason for hiding this comment

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

✨ PR Review

LGTM

Generated by LinearB AI and added by gitStream.
AI-generated content may contain inaccuracies. Please verify before using. We'd love your feedback! 🚀

@TamimiGitHub
Copy link
Member

Thanks @yan00126 ! I fixed a build issue from out backend and it looks good now. Thanks for your contribution

@TamimiGitHub TamimiGitHub merged commit 9d45cbc into master Jun 23, 2025
8 checks passed
@TamimiGitHub TamimiGitHub deleted the community-update branch July 22, 2025 20:07
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.

4 participants