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

Allow Teacher Leveling not just student #67

Open
fxpar opened this issue Jun 27, 2017 · 8 comments
Open

Allow Teacher Leveling not just student #67

fxpar opened this issue Jun 27, 2017 · 8 comments

Comments

@fxpar
Copy link

fxpar commented Jun 27, 2017

Hi,
It seems that only "students" events trigger some points.
Unfortunately, the staff is left out of the plugin "motivation reinforcement".
Could you allow also Teacher event like :

  • creating a course
  • adding an activity
  • Adding a question to the question bank
  • Assignment graded

Thank you for this plugin. (Sorry if I didn't post at the right place)
François

@FMCorz
Copy link
Owner

FMCorz commented Jun 27, 2017

Hi François,

This is the right place, and thanks for your suggestion. I agree that there is value in rewarding teachers, however that's not as simple as counting those events in. I can imagine that students would be confused to see their teachers competing with them on the ladder ;-).

I'll keep that in mind, perhaps another instance of the block?

Cheers,
Fred

@fxpar
Copy link
Author

fxpar commented Jun 27, 2017

Hi Fred,

Thank you for your answer.
And indeed, it might be better to have another instance for teachers...
(hadn't thought of that, of course)

Cheers
François

@fxpar
Copy link
Author

fxpar commented Jun 28, 2017

Hi,

I tweaked your module from xp to xpt (xp for teacher).
... seems to work with xp.
(Sorry, no clue on what to do with it on github or moodle plugin directory)

screen_20170628_095650

block_xpt_moodle33_2017032900.zip

@FMCorz
Copy link
Owner

FMCorz commented Jun 28, 2017

Thanks François. No doubt that would work, though I'm guessing there is a more elegant way to achieve this than duplicating the entire plugin. Though the plugin's architecture is oriented towards a single instance per course, perhaps there is a way to work it out.

@fxpar
Copy link
Author

fxpar commented Jun 28, 2017

;) Definitely not the subtle way, for sure ;)
Chose the easy / only way for me to do it.

Thanks again for this great plugin.
François

@fxpar
Copy link
Author

fxpar commented Jun 28, 2017

Some events don't show up though: forums for example.

@FMCorz
Copy link
Owner

FMCorz commented Jun 28, 2017

Hey, you'll have to dig a bit more :-D.

@fxpar
Copy link
Author

fxpar commented Jul 7, 2017

Hi,

Definitely didn't find the way to get hold of the whole list of events.

I tried to add the edulevel OTHER, but not much luck yet.
$infos['edulevel'] == \core\event\base::LEVEL_OTHER
(rule_event.php line 150; helper.php line 83)

I also erased the "isadmin" from the list of users we skip
(helper.php line 76)

Any idea to get me on the right track?
Have a nice week-end

Regards
François

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

No branches or pull requests

2 participants