Skip to content

chore: update office hours with direct link instead of the general calendar#1676

Draft
dtee1 wants to merge 6 commits intohiero-ledger:mainfrom
dtee1:update-office-hours-with-direct-link-instead-of-the-general-calendar
Draft

chore: update office hours with direct link instead of the general calendar#1676
dtee1 wants to merge 6 commits intohiero-ledger:mainfrom
dtee1:update-office-hours-with-direct-link-instead-of-the-general-calendar

Conversation

@dtee1
Copy link
Contributor

@dtee1 dtee1 commented Feb 2, 2026

Description:

Fixes #1634

Notes for reviewer:

Checklist

  • [✔ ] Documented (Code comments, README, etc.)
  • [ ✔] Tested (unit, integration, etc.)

dtee1 added 2 commits February 2, 2026 00:03
…alendar

Signed-off-by: dtee1 <davidtalson0@gmail.com>
Signed-off-by: dtee1 <davidtalson0@gmail.com>
Copilot AI review requested due to automatic review settings February 2, 2026 00:10
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 2, 2026

Walkthrough

This 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

Cohort / File(s) Summary
GitHub Automation Scripts
.github/scripts/bot-community-calls.sh, .github/scripts/bot-office-hours.sh
Removed CALENDAR_LINK variable declarations and updated comment body references to use MEETING_LINK instead, pointing to direct Zoom meeting URLs.
GitHub Workflow Scripts
.github/scripts/bot-workflows.js, .github/scripts/pr_inactivity_reminder.js
Updated meeting link constants from generic weekly calendar URL to direct Zoom meeting URLs with meeting IDs and passwords for Community Calls and Office Hours respectively.
Issue Template
.github/ISSUE_TEMPLATE/03_beginner_issue.yml
Replaced generic Python SDK Office Hour and Community Calls references with explicit clickable Zoom meeting links.
Documentation Files
docs/maintainers/good_first_issues_guidelines.md, docs/sdk_developers/training/workflow/11_submit_pull_request.md, docs/sdk_developers/workflow.md
Updated external meeting links from generic calendar views to direct Zoom meeting URLs with meeting IDs and passwords for Office Hours and Community Calls.
Changelog
CHANGELOG.md
Added entry documenting the Office Hours link update from generic calendar to direct meeting link.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description check ✅ Passed The PR description is related to the changeset, mentioning the link update and referencing the related issue #1634.
Linked Issues check ✅ Passed All generic calendar links have been replaced with appropriate direct meeting URLs across multiple files. The CHANGELOG entry was added, and no unrelated modifications were introduced.
Out of Scope Changes check ✅ Passed All changes are scoped to replacing generic calendar links with direct meeting URLs and updating documentation/scripts accordingly, with no unrelated modifications detected.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Title check ✅ Passed The title clearly and specifically describes the main change: replacing generic Office Hours calendar links with direct meeting links, which aligns with the primary objective across all modified files.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 2

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

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.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@exploreriii exploreriii changed the title Update office hours with direct link instead of the general calendar chore: update office hours with direct link instead of the general calendar Feb 2, 2026
@exploreriii exploreriii marked this pull request as draft February 2, 2026 02:25
Copy link
Contributor

@aceppaluni aceppaluni left a comment

Choose a reason for hiding this comment

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

@dtee1

This is looking good!!
The copilot suggestions are good. If you have any questions on implementing these changes let us know.

dtee1 and others added 3 commits February 2, 2026 12:21
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>
@github-actions
Copy link

github-actions bot commented Feb 2, 2026

[commit-verification-bot]
Hi, this is VerificationBot.
Your pull request cannot be merged as it has 1 unverified commit(s):

  • e5ea202 Merge branch 'hiero-ledger:main' into update-offic

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:
git commit -S -s -m "Your message here"

Thank you for contributing!

From the Hiero Python SDK Team

Signed-off-by: dtee1 <davidtalson0@gmail.com>
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

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.

Copy link
Contributor

@aceppaluni aceppaluni left a comment

Choose a reason for hiding this comment

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

@dtee1 This is looking good so far.

The AI suggestions are good. Please check you commits as e5ea202 is not verified.

Let us know if you have any questions, thank you!!

Copy link
Contributor

@exploreriii exploreriii left a comment

Choose a reason for hiding this comment

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

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

Copy link
Contributor

@exploreriii exploreriii left a comment

Choose a reason for hiding this comment

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

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!

@github-actions
Copy link

github-actions bot commented Feb 3, 2026

Hi, this is MergeConflictBot.
Your pull request cannot be merged because it contains merge conflicts.

Please resolve these conflicts locally and push the changes.

Quick Fix for CHANGELOG.md Conflicts

If your conflict is only in CHANGELOG.md, you can resolve it easily using the GitHub web editor:

  1. Click on the "Resolve conflicts" button in the PR
  2. Accept both changes (keep both changelog entries)
  3. Click "Mark as resolved"
  4. Commit the merge

For all other merge conflicts, please read:

Thank you for contributing!

Copy link
Contributor

@exploreriii exploreriii left a comment

Choose a reason for hiding this comment

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

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"

Image

@exploreriii exploreriii added the status: pending activity confirmation Developer needs to confirm they are interested in progressing with the PR label Feb 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

status: pending activity confirmation Developer needs to confirm they are interested in progressing with the PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Good First Issue]: Update Office Hours with direct link instead of the general calendar

3 participants