Add UTM enabled banner redirect#13724
Conversation
Signed-off-by: Muhammad Rizwan Munawar <muhammadrizwanmunawar123@gmail.com>
Signed-off-by: Muhammad Rizwan Munawar <muhammadrizwanmunawar123@gmail.com>
|
👋 Hello @RizwanMunawar, thank you for submitting a
For more guidance, please refer to our Contributing Guide. Don't hesitate to leave a comment if you have any questions. Thank you for contributing to Ultralytics! 🚀 |
UltralyticsAssistant
left a comment
There was a problem hiding this comment.
🔍 PR Review
Made with ❤️ by Ultralytics Actions
Good, minimal change overall, but both updated banner links currently use an invalid query-string format. Fixing the separators from repeated ? to & is necessary for the UTM tracking to work as intended.
💬 Posted 2 inline comments
Signed-off-by: Muhammad Rizwan Munawar <muhammadrizwanmunawar123@gmail.com>
Signed-off-by: Muhammad Rizwan Munawar <muhammadrizwanmunawar123@gmail.com>
UltralyticsAssistant
left a comment
There was a problem hiding this comment.
🔍 PR Review 2
Made with ❤️ by Ultralytics Actions
This PR looks clean. It makes a small, consistent documentation-only change in both README files, and I don't see any functional or maintainability issues in the diff.
|
Thanks — this looks like a clean docs-only update, and keeping both README variants aligned makes the change straightforward to maintain. |
🛠️ PR Summary
Made with ❤️ by Ultralytics Actions
🌟 Summary
Updates the README banner links to use a UTM-enabled redirect to the Ultralytics Platform for improved referral tracking 📈
📊 Key Changes
README.mdwith a UTM-tagged Ultralytics Platform URL.README.zh-CN.mdfor consistency across localized documentation.🎯 Purpose & Impact