FMCorz / moodle-block_xp

A gamification plugin for Moodle allowing students to gain experience points and level up.
https://levelup.plus/?ref=github
150 stars 41 forks source link

Hide register logs Issue #105 #107

Closed stellargela closed 3 years ago

FMCorz commented 4 years ago

Thank you for your this patch. Unfortunately we cannot change interfaces, so we won't be able to add the methods can_view_logs etc. I'm not certain how to best implement additional permissions within the existing framework, I'll have to think about it a bit.

stellargela commented 4 years ago

Hi! Thank you for your answer. I'm looking forward to your ideas and of course, if I can help in any way with the implementation, let me know!

goggo24 commented 4 years ago

@FMCorz have you found a solution? Do you have an idea? We'd love to use level up & level up pro in our insitution, but cannot use it unless the logs are visible for teachers :( We don't want to make a local patch!

FMCorz commented 4 years ago

Hi @goggo24,

We are unsure yet, we want to make sure that we do not bloat the plugin with individual features and are wondering whether this sort of privacy concern should be extended to other parts of the plugin at the same time. For instance, we had a request for the plugin to remove the links to user's profiles.

Is the only privacy concern of yours the ability for teachers to see how students earned their points? If the logs contained less information, would it work? The logs are useful for teachers to determine how the points are being earned and hiding them may make their work difficult. If the logs were anonymous, would it help?

Would you want to hide the logs as a site-wide option, or would there be courses in which the logs could be viewed by the teachers?

Are those teachers managing Level up!? In that, are they writing the rules, assigning points, etc?

Thank you for your input.

goggo24 commented 4 years ago

We would like to hide it site-wide. In our solution both cases are possible ;) We need to hide the logs, otherwise we cannot use this plugin and especially level up pro.

FMCorz commented 3 years ago

This has been implemented and will be included in the next release. Thank you for your feedback and contributions!