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

Add Deadline to Copy #207

Merged
merged 15 commits into from
Feb 7, 2024
Merged

Add Deadline to Copy #207

merged 15 commits into from
Feb 7, 2024

Conversation

qweliant
Copy link
Contributor

@qweliant qweliant commented Jan 24, 2024

Issue(s) Resolved

#202

Test Plan

Configure the integration with some number of days
send an evaluation invite to an email of your choice. As you move through the the pages be sure that:

  • the deadlines in the accept page and more information page are correct.
  • be sure that when you accept, the acceptance email has the correct deadline.
  • be sure that the eval page has has the correct dates.

Screenshots (if applicable)

Screenshot 2024-01-31 at 1 18 32 PM

Screenshot 2024-01-31 at 1 03 18 PM

Screenshot 2024-01-31 at 1 02 28 PM

Screenshot 2024-01-31 at 1 02 03 PM

Optional

Notes/Context/Gotchas

There are three deadlines.

  • Acceptance deadline - when to schedule another user to review
  • Nudge deadline - when to nudge the user to review
  • Submission deadline - when to remind user the date until submission is iver

Only the submission deadline is derived by input now. If we wanted to derive the nudge deadline from input we could add that to the config. Same with acceptance.

Supporting Docs

@qweliant qweliant changed the title update form schema and add deadline input field Add Deadline to Copy Jan 24, 2024
@qweliant qweliant linked an issue Jan 31, 2024 that may be closed by this pull request
@qweliant qweliant marked this pull request as ready for review January 31, 2024 18:59
@qweliant qweliant requested review from 3mcd and kalilsn January 31, 2024 19:02
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.

Show evaluation deadline
2 participants