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

[Documentation:Submission] Update gradeable creation #519

Merged
merged 11 commits into from
Aug 15, 2023
Merged

Conversation

IDzyre
Copy link
Contributor

@IDzyre IDzyre commented Aug 1, 2023

No description provided.

@IDzyre IDzyre changed the title [Documentation:Submissionupdate gradeable [Documentation:Submission] Update gradeable creation Aug 1, 2023
Copy link
Contributor

@NiaHeermance NiaHeermance left a comment

Choose a reason for hiding this comment

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

Awesome work! Just a couple of comments to fix. Also maybe you should delete the image we're no longer using from the repository?

manual data entry.

**_Student File Submission_** There are two types of gradeables where students can submit their files
to Submitty, directly uploading to the website, or by submitting the files using a version control system (VCS).
Copy link
Contributor

Choose a reason for hiding this comment

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

Should be ...their files to Submitty: directly...


If this is a team assignment, encourage the students to form their teams through the Submitty interface promptly, so they are not delayed by the creation of their team version control repository.

3. If the students files will be in a subdirectory of their main repository, select "yes" and put the path to the student's files in the required field.
Copy link
Contributor

Choose a reason for hiding this comment

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

Number says 1 on localhost, not 3. Numbering is broken here.

@NiaHeermance NiaHeermance self-requested a review August 9, 2023 16:34
Copy link
Contributor

@NiaHeermance NiaHeermance left a comment

Choose a reason for hiding this comment

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

Looks good!

@IDzyre IDzyre requested a review from bmcutler August 14, 2023 21:14
@bmcutler bmcutler merged commit d80e7f1 into main Aug 15, 2023
3 checks passed
@bmcutler bmcutler deleted the updateGradeable branch August 15, 2023 19:19
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.

3 participants