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

surveys: archive individual survey (fixes #6620) #7807

Open
wants to merge 13 commits into
base: master
Choose a base branch
from

Conversation

RheuX
Copy link
Member

@RheuX RheuX commented Nov 20, 2024

WIP for #6620 for number 2, archive/closing
Fixes #6620 number 2

  • figured out how to archive (there's only delete survey, mutugi mentioned something about it's by a tag, so checking if i can tag a survey for being closed/archived.
  • still displayed archived survey, but you can only export.
  • figured out a way to let users who still hasn't take the survey to not be able too anymore

probably update each users survey to add another key:value pair, so that I can check it on the end user side

just need to clean up the functions and logging statements, specifically for the forkJoin

Intended Behavior

You can now archive surveys,

Admin

  • when you archive surveys, you cannot edit, record, or send them anymore, but if anyone filled out the survey, you can still export them, and see it.

Learner/User

  • When a survey is archived, you cannot see it anymore, nor can you take it anymore

I can't fix, might need to be a separate PR

I cannot put a tooltip down when the buttons are disabled, since I made most of the buttons disabled for archive.

Also, you can only archive for surveys that you created, im not sure about surveys related to a courses (since I think courses on survey should always be open)

@RheuX RheuX added the WIP label Nov 20, 2024
@RheuX RheuX changed the title surveys: archive individual survey surveys: archive individual survey (fixes #6620) Nov 26, 2024
@RheuX RheuX removed the WIP label Nov 26, 2024
@RheuX RheuX marked this pull request as ready for review November 27, 2024 18:57
@RheuX RheuX requested a review from Gavinp14 December 12, 2024 21:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Surveys: Handle discarding sent surveys
3 participants