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

SC-124 Add evaluators link and status change funtionality to tasklist #2009

Merged
merged 2 commits into from
Dec 13, 2024

Conversation

serenaabbott11
Copy link
Contributor

@serenaabbott11 serenaabbott11 commented Dec 3, 2024

Changes in this PR

  • Adds the correct link to the add evaluators page
  • Changes the status to complete when there is an evaluator added
  • Adds back links to the pages
  • Adds validation for email format, and custom validation messages

Screen-shots or screen-capture of UI changes

Screenshot 2024-12-05 at 12 35 00

Copy link

github-actions bot commented Dec 3, 2024

Test coverage

Line: 87.2%
Branch: 77.25%

@serenaabbott11 serenaabbott11 force-pushed the sc-124-add-evaluators-link-to-tasklist branch from 30e9263 to 3f6fb59 Compare December 5, 2024 12:31
@serenaabbott11 serenaabbott11 changed the title Add evaluators link and status change funtionality to tasklist SC-124 Add evaluators link and status change funtionality to tasklist Dec 5, 2024
@serenaabbott11 serenaabbott11 self-assigned this Dec 5, 2024
Copy link

github-actions bot commented Dec 5, 2024

Test coverage

Line: 87.21%
Branch: 77.35%

@serenaabbott11 serenaabbott11 marked this pull request as ready for review December 5, 2024 12:41
@serenaabbott11 serenaabbott11 force-pushed the sc-124-add-evaluators-link-to-tasklist branch from 3f6fb59 to 4d709e2 Compare December 9, 2024 10:31
Copy link

github-actions bot commented Dec 9, 2024

Test coverage

Line: 87.21%
Branch: 77.35%

@serenaabbott11 serenaabbott11 force-pushed the sc-124-add-evaluators-link-to-tasklist branch from 4d709e2 to 4e8a441 Compare December 9, 2024 12:27
Copy link

github-actions bot commented Dec 9, 2024

Test coverage

Line: 87.24%
Branch: 77.32%

@serenaabbott11 serenaabbott11 force-pushed the sc-124-add-evaluators-link-to-tasklist branch from 4e8a441 to 9676e19 Compare December 10, 2024 14:31
Copy link

Test coverage

Line: 87.21%
Branch: 77.09%

Copy link

Test coverage

Line: 87.21%
Branch: 77.09%

@serenaabbott11 serenaabbott11 force-pushed the sc-124-add-evaluators-link-to-tasklist branch from 821f8a1 to 892bc67 Compare December 13, 2024 15:13
Copy link

Test coverage

Line: 87.04%
Branch: 76.98%

@serenaabbott11 serenaabbott11 merged commit fdac119 into main Dec 13, 2024
25 checks passed
@serenaabbott11 serenaabbott11 deleted the sc-124-add-evaluators-link-to-tasklist branch December 13, 2024 15:53
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