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 option to hide response to students in essay question #339

Open
wants to merge 1 commit into
base: MOODLE_39_STABLE
Choose a base branch
from

Conversation

adpe
Copy link

@adpe adpe commented May 11, 2021

Hi @mchurchward

We would really like to add a functionality to allow questionnaire creators to be able to hide a particular essay question. Therefore, users with a gradable role will not see the responses of such questions, as they serve only for internal purpose.

Do we want to use the extradata field, or shall we implement a new DB field?

Greets
Adrian

@adpe
Copy link
Author

adpe commented May 11, 2021

Here's a screenshot from my dev/testing stuff:

image

@adpe adpe force-pushed the feature/hide-essay-question-response branch 2 times, most recently from 3d480f4 to de07fb1 Compare June 2, 2021 14:42
@adpe
Copy link
Author

adpe commented Jun 9, 2021

Hi @mchurchward

Can you please respond my question? Depending on that I would invest more time to make it more general and as additional feature in the plugin. So that site-admins can enable/disable this feature. But the main question is, shall we add this to extradata or implement a new DB field?

Greets
Adrian

@adpe adpe force-pushed the feature/hide-essay-question-response branch 3 times, most recently from 1428abe to f0f8672 Compare June 11, 2021 13:20
@adpe adpe force-pushed the feature/hide-essay-question-response branch from f0f8672 to 3872be7 Compare June 11, 2021 13:26
@adpe
Copy link
Author

adpe commented Jun 14, 2021

Hi @mchurchward

We've improved the code already a bit. Another better generic solution would be:
image

Can you please answer if you want to see this PR merged in the plugin or if it's not realistic?

Greets
Adrian

@mchurchward
Copy link
Contributor

Sorry for the delay. I have been a bit too busy lately. I will review this soon.

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.

2 participants