Skip to content

CTP-6662: Fix submission button not visible when personal deadline exists. - #310

Open
cwarwicker wants to merge 1 commit into
ucl-isd:mainfrom
cwarwicker:CTP-6662
Open

CTP-6662: Fix submission button not visible when personal deadline exists.#310
cwarwicker wants to merge 1 commit into
ucl-isd:mainfrom
cwarwicker:CTP-6662

Conversation

@cwarwicker

Copy link
Copy Markdown
Collaborator

No description provided.

Copilot AI lite review requested due to automatic review settings August 12, 2026 14:49

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR addresses a permission/visibility issue in mod_coursework where students with an active personal deadline could not see the “Upload your submission” action after the main coursework deadline had passed.

Changes:

  • Adds an ability rule to allow new submissions when a student has a personal deadline in the future.
  • Adds a Behat scenario to cover submitting after the default deadline when a personal deadline exists.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
classes/ability.php Adds a new permission rule to allow “new submission” when a valid personal deadline exists.
tests/behat/deadline.feature Adds a UI scenario asserting the submission action is visible when a personal deadline is present.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread classes/ability.php
Comment thread tests/behat/deadline.feature Outdated
Copilot AI review requested due to automatic review settings August 13, 2026 08:22

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 2 out of 2 changed files in this pull request and generated no new comments.

@cwarwicker cwarwicker self-assigned this Aug 13, 2026
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