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 API/CLI endpoint for attachments #2035

Closed
damianhxy opened this issue Dec 17, 2023 · 1 comment · Fixed by #2144
Closed

Add API/CLI endpoint for attachments #2035

damianhxy opened this issue Dec 17, 2023 · 1 comment · Fixed by #2144

Comments

@damianhxy
Copy link
Member

damianhxy commented Dec 17, 2023

Issues with current implementation of attachments (as per #1983)

  • does not support static links (tedious to update existing permalinks)
  • no easy way to mass-import

For that reason, using the assessment "direct-download" feature (#2020) is somewhat more convenient for instructors

  • only course name changes, easy to update links
  • can mass import assessments

Potential solution

  • Add an endpoint which uploads an attachment and returns the attachment link -> easy to generate HTML lists
  • Rename "attachments" to "course materials"

Also possible: allow the specification of a canonical name

Note: An ad-hoc workaround was available in #2020 #2021 but this would be a longer-term solution.

@damianhxy
Copy link
Member Author

damianhxy commented Apr 16, 2024

Update as of Apr 15 2024: rather than going down this route, we could simply annotate assessments with a "writeup" label if one exists, and that downloads the writeup upon clicking.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant