illinois-cs241 / broadway-on-demand

Webapp that facilitates on-demand assignment autograding via Broadway.
Other
11 stars 4 forks source link

Slack Integration for Broadway Grading #21

Open bhuvy2 opened 5 years ago

bhuvy2 commented 5 years ago

It'd be nice to have an api endpoint where members of the admin team can trigger runs through slack. The slack facing api could be /broadway grade <assign> [netid]. The slackbot should return that the run started.

I don't know if this should be the same issue, but it'd be cool if the bot eventually posted a message to the channel with that person's results (or maybe a different slack command like /broadway status <id>

shreyas208 commented 5 years ago

Useful resources:

We need to get the user's Slack UID from the incoming slash command payload and then use Slack Web API to get their email address.

shreyas208 commented 5 years ago

Possible formats:

/broadway grade sp19-cs241 malloc "netid1, netid2, netid3" (must return run_id) /broadway grade-all sp19-cs241 malloc /broadway status <run_id>

bhuvy2 commented 5 years ago

Should we have a commit-run (i.e. one that just runs a student and one that commits results?)

shreyas208 commented 5 years ago

We could have a flag on grade and grade-all that flips between commit to student and don't commit to student? (If no commit, we need to find an alternate feedback mechanism. If/when feedback is coming back to on-demand, this will be resolved)

bhuvy2 commented 5 years ago

I guess the question I was trying to ask is what is the main goal of the slack integration? Is it for us diagnostically or to generate one off runs for other cases.

shreyas208 commented 5 years ago

You tell me :) It would be nice to have it for both purposes, which I think should be the goal for this feature.