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

'What do you think?' Feedback prompt - Any way to get rid of it? #5104

Closed
Aypak opened this Issue Apr 26, 2016 · 6 comments

Comments

Projects
None yet
3 participants
@Aypak
Contributor

Aypak commented Apr 26, 2016

I am running ka-lite 0.16.2 on a rpi 2 and accessing it using android tablets. The prompt asking users to rate the quality and difficulty of the content appears everytime a student attempts an exercise. I understand that feedback is important to the ka-lite team. However, I think it takes up too much space on the small screen of the tablet, and also distracts students as it appears before every exercise. Is there any way to disable or hide this feedback prompt for students?

@benjaoming

This comment has been minimized.

Show comment
Hide comment
@benjaoming

benjaoming May 2, 2016

Member

Thanks for the feedback, @Aypak

I think this sounds like a reasonable feature. Something that should be configurable as a project setting. Please see #4054 for a description about app-based settings.

Setting something KALITE_DISABLE_USER_FEEDBACK = True in ~/.kalite/settings.py would be a good solution.

If you're willing to contribute this, we would definitely have a look.

Member

benjaoming commented May 2, 2016

Thanks for the feedback, @Aypak

I think this sounds like a reasonable feature. Something that should be configurable as a project setting. Please see #4054 for a description about app-based settings.

Setting something KALITE_DISABLE_USER_FEEDBACK = True in ~/.kalite/settings.py would be a good solution.

If you're willing to contribute this, we would definitely have a look.

@Aypak

This comment has been minimized.

Show comment
Hide comment
@Aypak

Aypak May 9, 2016

Contributor

Hi @benjaoming, thank you for your response. I'm sorry for the late reply. I currently have scanty knowledge of django and thus I don't know how I can contribute to this. I will follow along and try to learn as much as i can. I'll create a pull request if i arrive at a solution.
Thanks!

Contributor

Aypak commented May 9, 2016

Hi @benjaoming, thank you for your response. I'm sorry for the late reply. I currently have scanty knowledge of django and thus I don't know how I can contribute to this. I will follow along and try to learn as much as i can. I'll create a pull request if i arrive at a solution.
Thanks!

@radinamatic

This comment has been minimized.

Show comment
Hide comment
@radinamatic

radinamatic May 9, 2016

Contributor

Or maybe activating the rating only after the learner completed the "8 out of 10" exercises? Rating would be based on a more complete learning experience...

Contributor

radinamatic commented May 9, 2016

Or maybe activating the rating only after the learner completed the "8 out of 10" exercises? Rating would be based on a more complete learning experience...

@benjaoming

This comment has been minimized.

Show comment
Hide comment
@benjaoming

benjaoming May 9, 2016

Member

@radinamatic I think that's a feature that's more appropriate for Kolibri :)

Member

benjaoming commented May 9, 2016

@radinamatic I think that's a feature that's more appropriate for Kolibri :)

@radinamatic

This comment has been minimized.

Show comment
Hide comment
@radinamatic

radinamatic May 9, 2016

Contributor

You're reading my mind @benjaoming!
I was about to suggest it as a feature for Kolibri and than got distracted... 😃

Contributor

radinamatic commented May 9, 2016

You're reading my mind @benjaoming!
I was about to suggest it as a feature for Kolibri and than got distracted... 😃

benjaoming added a commit to benjaoming/ka-lite that referenced this issue Oct 2, 2016

benjaoming added a commit to benjaoming/ka-lite that referenced this issue Oct 2, 2016

@benjaoming benjaoming referenced this issue Oct 2, 2016

Merged

Add setting HIDE_CONTENT_RATING #5312

0 of 3 tasks complete

@benjaoming benjaoming self-assigned this Oct 2, 2016

@benjaoming benjaoming added the has PR label Oct 2, 2016

benjaoming added a commit to benjaoming/ka-lite that referenced this issue Oct 2, 2016

benjaoming added a commit to benjaoming/ka-lite that referenced this issue Oct 2, 2016

benjaoming added a commit to benjaoming/ka-lite that referenced this issue Oct 2, 2016

benjaoming added a commit to benjaoming/ka-lite that referenced this issue Oct 2, 2016

benjaoming added a commit to benjaoming/ka-lite that referenced this issue Oct 3, 2016

benjaoming added a commit to benjaoming/ka-lite that referenced this issue Oct 3, 2016

@benjaoming

This comment has been minimized.

Show comment
Hide comment
@benjaoming

benjaoming Oct 3, 2016

Member

Sorry I had some issues getting this to work and kept force-pushing new commits on top of the old one :)

But it's fixed now @Aypak

Member

benjaoming commented Oct 3, 2016

Sorry I had some issues getting this to work and kept force-pushing new commits on top of the old one :)

But it's fixed now @Aypak

@benjaoming benjaoming closed this Oct 3, 2016

@benjaoming benjaoming removed the has PR label Oct 3, 2016

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