BF: Don't check for assignments if Clever or ClassLink user is not a student

richford commented 3 months ago

Proposed changes

This PR fixes an issue where Clever and ClassLink teachers had to wait indefinitely to check for non-existent assignments.

What types of changes does this pull request introduce?

[x] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Refactoring (non-breaking change that does not add functionality but makes code cleaner or more efficient)
[ ] Documentation Update
[ ] Tests (new or updated tests)
[ ] Style (changes to code styling)
[ ] CI (continuous integration changes)
[ ] Repository Maintenance
[ ] Other (please describe below)

[x] I have read the guidelines for contributing.
[x] The changes in this PR are as small as they can be. They represent one and only one fix or enhancement.
[x] Linting checks pass with my changes.
[x] Any existing unit tests pass with my changes.
[x] Any existing end-to-end tests pass with my changes.
[ ] I have added tests that prove my fix is effective or that my feature works.
[ ] If this PR fixes an existing issue, I have added a unit or end-to-end test that will detect if this issue reoccurs.
[x] I have added JSDoc comments as appropriate.
[x] I have added the necessary documentation to the roar-docs repository.
[x] I have shared this PR on the roar-pr-reviews channel (if I have access)
[x] I have linked relevant issues (if any)

We are not currently able to test Clever or ClassLink integration in CI environments.

github-actions[bot] commented 3 months ago

Visit the preview URL for this PR (updated for commit 00e906c):

_{(expires Mon, 01 Jul 2024 21:02:17 GMT)}

_{Sign: 2631e9c58fd0104ecbfddd72a62245ddac467460}

ROAR

cypress[bot] commented 3 months ago

0

26

0

Details:

Tests for PR 687 "BF: Don't check for assignments if Clever or ClassLink user is...
Project: roar-dashboard-e2e	Commit: `00e906ca7f`
Status: Passed	Duration: 03:31 💡
Started: Jun 24, 2024 9:02 PM	Ended: Jun 24, 2024 9:06 PM