Skip to content

Conversation

arora-r
Copy link
Member

@arora-r arora-r commented Sep 2, 2025

PR Description

Resolves issue #17

Overview:

Type of Issue:

  • Feature (feat): New functionality or feature added.
  • Bug Fix (bug): Issue or bug resolved.
  • Chore (chore): Maintenance, refactoring, or non-functional changes.
  • Documentation Update (doc): Documentation improvements or additions.

Change Type:

  • Major: Significant changes that introduce new features, large refactoring, or breaking changes. Requires thorough review and testing.
  • Minor: Small to medium changes, such as adding new functionality that is backward-compatible or minor refactoring. Moderate review needed.
  • Patch: Bug fixes, small tweaks, or documentation updates. Light review is sufficient.

Testing & Validation:

  • Unit Tests: Added/updated to cover new logic or edge cases.
  • Integration Tests: Updated to verify interactions between components.
  • E2E Tests: Performed end-to-end testing in staging or development environment.
  • Manual Testing: The changes were manually tested and validated.
  • No Regressions: Verified that no existing functionality is broken.

Purpose:

This PR handles adding the ability for admins to set cooldown periods.

Context:

#17

Basic Usage:

Admins can set a cooldown period and when the period starts (after a certain amount of attempts). Once this threshold is reached for learners, they will have to wait a cooldown period to re-attempt an assignment.

Notes to Reviewer:

Copy link
Collaborator

@MmagdyHafezZ MmagdyHafezZ left a comment

Choose a reason for hiding this comment

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

LGTM

@MmagdyHafezZ MmagdyHafezZ merged commit 8b0aa54 into master Oct 9, 2025
9 checks passed
@MmagdyHafezZ MmagdyHafezZ deleted the feat/include-cooldown-periods branch October 9, 2025 10:27
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.

2 participants