Skip to content
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

[Test failure] TC_AUTHOR_121: The grace period cannot be set to less than 1 hour. #365

Closed
BryanttV opened this issue May 27, 2024 · 4 comments
Labels
redwood release testing Affects the upcoming release (attention needed)

Comments

@BryanttV
Copy link

BryanttV commented May 27, 2024

Release

Redwood

Expected behavior

As a course author, I want to set a grace period at the course level so students can submit assignments after their due date.

Actual behavior

As a course author, I cannot set a grace period of less than one hour, because the value I type is automatically deleted, even if the format is correct.

Steps to reproduce

  1. From Studio go to Settings > Grading
  2. On the Grading page, under Grading Rules & Policies, enter a value in the Grace Period on Deadline field.
  3. Try typing for example: 00:30. The value will be deleted automatically.
TC_AUTHOR_121.mp4

Additional information

The functionality works correctly by setting a value equal to or greater than 1 hour.

@BryanttV BryanttV added needs triage release testing Affects the upcoming release (attention needed) labels May 27, 2024
@BryanttV BryanttV changed the title [Test failure] TC_AUTHOR_121: The grace period cannot be set to less than 1 hour. [Test failure] TC_AUTHOR_121: The grace period cannot be set to less than 1 hour. May 27, 2024
@mariajgrimaldi mariajgrimaldi changed the title [Test failure] TC_AUTHOR_121: The grace period cannot be set to less than 1 hour. [Test failure] TC_AUTHOR_121: The grace period cannot be set to less than 1 hour. May 30, 2024
@mariajgrimaldi
Copy link
Member

mariajgrimaldi commented May 31, 2024

Thanks for the report! I confirmed this is happening in the redwood sandbox, so I opened an issue in the course authoring MFE for maintainers to review.

@mariajgrimaldi
Copy link
Member

I created a backport for the Redwood branch; I hope it gets merged early this week. Thanks!

@mariajgrimaldi
Copy link
Member

Backport merged! I'll close this issue once we test it in the redwood sandbox.

@mariajgrimaldi
Copy link
Member

I can confirm that I can set the grace period in the sandbox to less than an hour.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
redwood release testing Affects the upcoming release (attention needed)
Projects
Development

No branches or pull requests

2 participants