feat(user): Add answer-creating functionality

hfaran / piazza-api

Unofficial Client for Piazza's Internal API

https://hfaran.github.io/posts/reverse-engineering-piazzas-api/

MIT License

185 stars 54 forks source link

feat(user): Add answer-creating functionality #31

Closed DavidSnider closed 8 years ago

DavidSnider commented 8 years ago

I added the ability to create an answer to a post. It's based on work done by @arxanas on followup creation, and the results are fairly similar, except for answers rather than followups.

arxanas commented 8 years ago

Can this toggle between student and instructor answers? What is the behavior about that right now? (I'm guessing i_answer means instructor answer?)

DavidSnider commented 8 years ago

You make a good point, I'm going to change this to clarify instructor answer, which is what this deals with.

GitCop commented 8 years ago

There were the following issues with your Pull Request

Commit: 04aa9e8926f41ae4915280134a4c53963f5e6424
- Commits must be in the following format: %{type}(%{scope}): %{description}

Guidelines are available at https://github.com/hfaran/piazza-api

This message was auto-generated by https://gitcop.com

hfaran commented 8 years ago

Looks good to me! Has the point @arxanas brought up been resolved or are you still addressing that?

DavidSnider commented 8 years ago

I changed the code so it's clear that this is for instructor's responses. I didn't do student responses as I didn't have any easy way to test and I couldn't think of a good reason I'd want to be automating student responses.

hfaran commented 8 years ago

Yeah, that makes sense. I will go ahead and merge this then.

arxanas commented 8 years ago

Can we get this on PyPI?

hfaran commented 8 years ago

@arxanas Sure thing. I'll make a new release.

hfaran commented 8 years ago

0.5.2 has been released and pushed to PyPI. @DavidSnider I've added you to the contributors list. :)