chore: update office hours with direct link instead of the general calendar#1676
Conversation
…alendar Signed-off-by: dtee1 <davidtalson0@gmail.com>
Signed-off-by: dtee1 <davidtalson0@gmail.com>
WalkthroughThis pull request replaces generic Office Hours and Community Calls calendar links across the repository with direct Zoom meeting URLs including meeting IDs and passwords. Changes span GitHub automation scripts, documentation, an issue template, and a changelog entry. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Pull request overview
Updates documentation and bot messaging to use direct Zoom meeting links for Office Hours and Community Calls (instead of the generic Hiero calendar), addressing #1634.
Changes:
- Replaced generic calendar URLs with direct meeting URLs across contributor docs and GitHub automation scripts.
- Added a changelog entry describing the update.
- Updated issue template guidance to point contributors to the direct meeting links.
Reviewed changes
Copilot reviewed 9 out of 9 changed files in this pull request and generated 6 comments.
Show a summary per file
| File | Description |
|---|---|
| docs/sdk_developers/workflow.md | Updates Office Hours / Community Calls help links to direct meeting URLs. |
| docs/sdk_developers/training/workflow/11_submit_pull_request.md | Updates Office Hours link used when checks fail to the direct meeting URL. |
| docs/maintainers/good_first_issues_guidelines.md | Updates Community Calls link to the direct meeting URL. |
| CHANGELOG.md | Adds an UNRELEASED changelog entry for the link update. |
| .github/scripts/pr_inactivity_reminder.js | Updates inactivity reminder’s Office Hours link to the direct meeting URL. |
| .github/scripts/bot-workflows.js | Updates WorkflowBot’s Community Calls link to the direct meeting URL. |
| .github/scripts/bot-office-hours.sh | Removes calendar link and updates disclaimer link to meeting URL. |
| .github/scripts/bot-community-calls.sh | Removes calendar link and updates disclaimer link to meeting URL. |
| .github/ISSUE_TEMPLATE/03_beginner_issue.yml | Updates contributor-facing template text to include the direct meeting URLs. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
aceppaluni
left a comment
There was a problem hiding this comment.
This is looking good!!
The copilot suggestions are good. If you have any questions on implementing these changes let us know.
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com> Signed-off-by: David Talson <37641675+dtee1@users.noreply.github.com>
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com> Signed-off-by: David Talson <37641675+dtee1@users.noreply.github.com>
…-link-instead-of-the-general-calendar
|
[commit-verification-bot]
View your commit verification status: Commits Tab. To achieve verified status, please read: Remember, you require a GPG key and each commit must be signed with: Thank you for contributing! From the Hiero Python SDK Team |
Signed-off-by: dtee1 <davidtalson0@gmail.com>
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 10 out of 10 changed files in this pull request and generated 5 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
exploreriii
left a comment
There was a problem hiding this comment.
Hi @dtee1
Really, really excellent job thank you for being so thorough!
AI did point out a couple of other good to haves - those are out of scope of the issue description, I will create them as separate good first issues 👍
Many thanks again
There was a problem hiding this comment.
For now unfortunately there is one commit that was not signed correctly, which you can see here
https://github.com/hiero-ledger/hiero-sdk-python/pull/1676/commits
Please can you do:
git reset --soft HEAD~6
then re-commit
git commit -S -s -m "chore: update office hour and community call links"
Then we can merge this. Thanks!
|
Hi, this is MergeConflictBot. Please resolve these conflicts locally and push the changes. Quick Fix for CHANGELOG.md ConflictsIf your conflict is only in CHANGELOG.md, you can resolve it easily using the GitHub web editor:
For all other merge conflicts, please read: Thank you for contributing! |
exploreriii
left a comment
There was a problem hiding this comment.
Hi @dtee1
We are unable to accept this PR until it meets the signing requirements
All but one of your commit is signed correctly
I would recommend
git reset --soft HEAD~6
then resign them again
git commit -S -s -m "chore: update office hour link"
Description:
Fixes #1634
Notes for reviewer:
Checklist