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

Development: Add student code of conduct and apso as a static assets #7390

Merged
merged 7 commits into from
Oct 18, 2023

Conversation

jakubriegel
Copy link
Contributor

@jakubriegel jakubriegel commented Oct 16, 2023

Checklist

General

Motivation and Context

The student code of conduct linked in the template of plagiarism case post and linked in the post created by continuous plagiarism control was no longer available.

Original thread on Slack: https://ls1tum.slack.com/archives/G6TCVL6HL/p1696873741965929.

Description

The student code of conduct is now a static asset of Artemis.

Steps for Testing

Prerequisites:

  1. Log in to Artemis.
  2. Go to TEST_SERVER_URL/content/documents/student-code-of-conduct.pdf and check if the PDF loads.

Review Progress

Code Review

  • Code Review 1
  • Code Review 2

Manual Tests

  • Test 1
  • Test 2

@jakubriegel jakubriegel self-assigned this Oct 16, 2023
@github-actions github-actions bot added tests server Pull requests that update Java code. (Added Automatically!) client Pull requests that update TypeScript code. (Added Automatically!) database Pull requests that update the database. (Added Automatically!). Require a CRITICAL deployment. config-change Pull requests that change the config in a way that they require a deployment via Ansible. labels Oct 16, 2023
@jakubriegel jakubriegel changed the base branch from develop to plagiarism-detection-cpc October 16, 2023 13:13
@jakubriegel jakubriegel changed the title Development: Development: Add student code of conduct as a static asset Oct 16, 2023
@jakubriegel jakubriegel removed tests database Pull requests that update the database. (Added Automatically!). Require a CRITICAL deployment. config-change Pull requests that change the config in a way that they require a deployment via Ansible. labels Oct 16, 2023
@jakubriegel jakubriegel marked this pull request as ready for review October 16, 2023 13:36
@jakubriegel jakubriegel requested a review from a team as a code owner October 16, 2023 13:36
Copy link
Contributor

@MichaelOwenDyer MichaelOwenDyer 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!

@jakubriegel jakubriegel changed the title Development: Add student code of conduct as a static asset Development: Add student code of conduct and apso as a static assets Oct 16, 2023
@Strohgelaender
Copy link
Contributor

Ping @b-fein @dfuchss you should probably replace these files in your local instance / fork.

Copy link
Contributor

@b-fein b-fein left a comment

Choose a reason for hiding this comment

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

The assets should be located in src/main/resources/public like the logo.

Then the existing customisation steps (https://ls1intum.github.io/Artemis/dev/setup/#customize-your-artemis-instance) can be documented and applied as well.

The links to the resources should then be relative to link to the ones of the current Artemis instance rather than the ones from TUM.

Copy link
Contributor

@aplr aplr 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, w/ the changes suggested by @b-fein.

@dfuchss
Copy link
Contributor

dfuchss commented Oct 16, 2023

I can look at it next week

docs/dev/setup.rst Outdated Show resolved Hide resolved
Co-authored-by: Benedikt Fein <[email protected]>
Copy link
Contributor

@b-fein b-fein left a comment

Choose a reason for hiding this comment

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

Thanks! Code and documentation for the customisation looks good now.

Copy link
Contributor

@lennart-keller lennart-keller left a comment

Choose a reason for hiding this comment

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

documentation lgtm

@jakubriegel
Copy link
Contributor Author

This PR is stacked and should be merged after #7302 is merged

@jakubriegel jakubriegel added the stacked-pr PR that depends on another PR label Oct 17, 2023
@krusche krusche merged commit b28a81a into plagiarism-detection-cpc Oct 18, 2023
30 of 36 checks passed
@krusche krusche deleted the code-of-conduct-static-file branch October 18, 2023 06:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
client Pull requests that update TypeScript code. (Added Automatically!) documentation ready to merge server Pull requests that update Java code. (Added Automatically!) small stacked-pr PR that depends on another PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants