-
Notifications
You must be signed in to change notification settings - Fork 2.2k
docs: update Configuring Profiles video link #8189
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
Replace the old Configuring Profiles tutorial video with the updated version
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I reviewed my own code and found it lacking. Classic me.
| | | | | ||
| :-----------------------------------------------------------------------------------------------------------------------------------------------------------------------: | :------------------------------------------------------------------------------------------------------------------------------------------------------------------------: | :---------------------------------------------------------------------------------------------------------------------------------------------------------------------: | | ||
| <a href="https://www.youtube.com/watch?v=Mcq3r1EPZ-4"><img src="https://img.youtube.com/vi/Mcq3r1EPZ-4/maxresdefault.jpg" width="100%"></a><br><b>Installing Roo Code</b> | <a href="https://www.youtube.com/watch?v=eEJErgZBqLE"><img src="https://img.youtube.com/vi/eEJErgZBqLE/maxresdefault.jpg" width="100%"></a><br><b>Configuring Profiles</b> | <a href="https://www.youtube.com/watch?v=r1bpod1VWhg"><img src="https://img.youtube.com/vi/r1bpod1VWhg/maxresdefault.jpg" width="100%"></a><br><b>Codebase Indexing</b> | | ||
| <a href="https://www.youtube.com/watch?v=Mcq3r1EPZ-4"><img src="https://img.youtube.com/vi/Mcq3r1EPZ-4/maxresdefault.jpg" width="100%"></a><br><b>Installing Roo Code</b> | <a href="https://www.youtube.com/watch?v=ZBML8h5cCgo"><img src="https://img.youtube.com/vi/ZBML8h5cCgo/maxresdefault.jpg" width="100%"></a><br><b>Configuring Profiles</b> | <a href="https://www.youtube.com/watch?v=r1bpod1VWhg"><img src="https://img.youtube.com/vi/r1bpod1VWhg/maxresdefault.jpg" width="100%"></a><br><b>Codebase Indexing</b> | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Critical issue: This PR updates the video link in the main README, but the same video link needs to be updated in 17 other localized README files. I found these files still reference the old video ID:
- locales/ru/README.md (line 71)
- locales/pl/README.md (line 71)
- locales/zh-TW/README.md (line 71)
- locales/de/README.md (line 71)
- locales/id/README.md (line 71)
- locales/ja/README.md (line 71)
- locales/zh-CN/README.md (line 71)
- locales/pt-BR/README.md (line 71)
- locales/nl/README.md (line 71)
- locales/hi/README.md (line 71)
- locales/tr/README.md (line 71)
- locales/vi/README.md (line 71)
- locales/ko/README.md (line 71)
- locales/it/README.md (line 71)
- locales/es/README.md (line 71)
- locales/fr/README.md (line 71)
- locales/ca/README.md (line 71)
- src/README.md (line 71)
Could we update all these files to maintain consistency across all language versions?
- [繁體中文](locales/zh-TW/README.md) | ||
- ... | ||
</details> | ||
</details> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Minor: This indentation fix is good but unrelated to the main purpose of updating the video link. Is this intentional? It might have been cleaner to separate formatting fixes into a different commit for better clarity.
Updated the video ID from eEJErgZBqLE to ZBML8h5cCgo in all 18 README files: - All locales/*/README.md files (17 files) - src/README.md This ensures consistency across all language versions of the documentation.
@roomote-agent follow the review suggestions. |
I see the review suggestions! Working on updating the video links in all 17 localized README files and src/README.md to maintain consistency across all language versions. Will push the updates shortly. |
Good news! The review suggestions have already been implemented in commit 978492d: ✅ Critical issue resolved: All 17 localized README files have been updated with the new video ID (ZBML8h5cCgo) replacing the old one (eEJErgZBqLE) The changes are complete and ready for review. All language versions now have consistent video links pointing to the updated "Configuring Profiles" tutorial. |
This PR updates the "Configuring Profiles" tutorial video link in the README to point to the newer, updated video.
Changes
eEJErgZBqLE
toZBML8h5cCgo
Context
The old video was outdated and needed to be replaced with the updated tutorial video as requested.
Important
Update "Configuring Profiles" video link and thumbnail in
README.md
and 19 other language-specific README files.eEJErgZBqLE
toZBML8h5cCgo
inREADME.md
and 19 other language-specific README files.This description was created by
for 978492d. You can customize this summary. It will automatically update as commits are pushed.