Skip to content

add timedelta for tgbl-mooc#393

Merged
Jacob-Chmura merged 2 commits intotgm-team:mainfrom
databaaz:databaaz/tgbl_mooc_td
Jan 22, 2026
Merged

add timedelta for tgbl-mooc#393
Jacob-Chmura merged 2 commits intotgm-team:mainfrom
databaaz:databaaz/tgbl_mooc_td

Conversation

@databaaz
Copy link
Contributor

Summary / Description

Provide a brief overview of what this PR does, and any relevant context.

Related Issues: #392

Type of Change

  • Bug fix

Test Evidence

Describe how this PR has been tested.

  • Unit tests

Questions / Discussion Points

List any areas where you’d like reviewer input or have open questions.

Copy link
Collaborator

@shenyangHuang shenyangHuang left a comment

Choose a reason for hiding this comment

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

approved

@benjaminnNgo benjaminnNgo self-requested a review January 22, 2026 13:18
Copy link
Contributor

@benjaminnNgo benjaminnNgo left a comment

Choose a reason for hiding this comment

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

Hi @databaaz,

Thank you for the PR. Looks good to me. Before merging, please make sure to update unit test here

@codecov
Copy link

codecov bot commented Jan 22, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@Jacob-Chmura Jacob-Chmura merged commit 839c55e into tgm-team:main Jan 22, 2026
7 checks passed
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