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

Slack Integration for Broadway Grading #21

Open
bhuvy2 opened this issue Jan 25, 2019 — with Slack · 6 comments
Open

Slack Integration for Broadway Grading #21

bhuvy2 opened this issue Jan 25, 2019 — with Slack · 6 comments
Assignees
Labels
enhancement New feature or request

Comments

Copy link

bhuvy2 commented Jan 25, 2019

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>

@bhuvy2 bhuvy2 added the enhancement New feature or request label Jan 25, 2019 — with Slack
@shreyas208
Copy link
Collaborator

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
Copy link
Collaborator

shreyas208 commented Jan 26, 2019

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
Copy link
Author

bhuvy2 commented Jan 26, 2019

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

@shreyas208
Copy link
Collaborator

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
Copy link
Author

bhuvy2 commented Jan 26, 2019

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
Copy link
Collaborator

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants