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.

Sign up for GitHub

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

Jump to bottom

Add event handler method #29

Closed
rwblickhan opened this issue Oct 6, 2018 · 0 comments
Closed

Add event handler method #29

rwblickhan opened this issue Oct 6, 2018 · 0 comments
Assignees
@rmcreyes
Labels
theme:feature Request a new feature.
Projects
Rocket 2 Planning

Comments

@rwblickhan
Copy link
Collaborator

Add a core.py (or some other reasonable name) to a command/ directory (or some other reasonable name) that will expose an event handler method that will be sent the event JSON strings from the Slack client. For now it just has to accept these and deserialize them to into SlackEvent objects (as implemented in #28).
@rwblickhan rwblickhan added the theme:feature Request a new feature. label Oct 6, 2018
@rwblickhan rwblickhan added this to Backlog/feature requests in Rocket 2 Planning via automation Oct 6, 2018
@rwblickhan rwblickhan mentioned this issue Oct 6, 2018
Closed
@rwblickhan rwblickhan moved this from Backlog/feature requests to To do in Rocket 2 Planning Oct 6, 2018
@rwblickhan rwblickhan mentioned this issue Oct 6, 2018
Closed
@rmcreyes rmcreyes moved this from To do to In progress in Rocket 2 Planning Oct 12, 2018
rmcreyes pushed a commit that referenced this issue Oct 21, 2018
Merge branch 'master' of https://github.com/ubclaunchpad/rocket2.0 in…
f0440fb 
…to rmcreyes/#29-event-handler
rmcreyes pushed a commit that referenced this issue Oct 30, 2018
Merge branch 'master' of https://github.com/ubclaunchpad/rocket2.0 in…
0ae655b 
…to rmcreyes/#29-event-handler
@rmcreyes rmcreyes mentioned this issue Oct 30, 2018
Merged
@rmcreyes rmcreyes moved this from In progress to Needs review in Rocket 2 Planning Oct 30, 2018
chuck-sys pushed a commit that referenced this issue Nov 3, 2018
Merge pull request #68 from ubclaunchpad/rmcreyes/#29-event-handler
52c0f66 
Rmcreyes/#29 event handler
@rmcreyes rmcreyes moved this from Needs review to Done in Rocket 2 Planning Nov 3, 2018
@rmcreyes rmcreyes closed this as completed Nov 3, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@rmcreyes rmcreyes

Labels
theme:feature Request a new feature.
Projects
Rocket 2 Planning
✅ Done
Milestone
No milestone
Development

No branches or pull requests
2 participants
@rwblickhan @rmcreyes