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

Develop #66

Merged
merged 24 commits into from
Nov 8, 2020
Merged

Develop #66

merged 24 commits into from
Nov 8, 2020

Conversation

taidopurason
Copy link
Collaborator

Merging into master. end of iteration 3

taidopurason and others added 24 commits October 14, 2020 14:26
… not seeing any tests in other Xatkit platforms when it comes to sent messages, tests have not yet been written. See #21 #58
…y for the bot developer to create them (still needs work). Implemented Postback Button, however the postback mechanic itself has not been implemented. See #56 #57 #33
…IT DOESN'T WORK. Facebook keeps bugging me about the message parameter being necessary, even though it is clearly present. I don't know what I'm doing wrong, please help and notify me :(
…achment-ids back, as well as tried to implement a secondary option of sending files directly. Netiher works atm, and my desire to get in touch with Facebook support is increasing. For example, when sending the file with attachment_id, it now gives the error of "wrong number of files sent", even though it doesn't make sense. Such an error should already appear earlier and has. The other method also doesn't work and is telling me that the recipient parameter is required, again, it's there. This leads me to believe that I don't either 1) understand how the Facebook curl examples work or 2) how using the JsonRestRequest in the xatkit rest platform package works.
…t work. Needs fixing. Alternatively add recipent id to the file uploaded directly, not upload and send with attachment_id
…ution. A proper solution needs heavy refactoring, so sorry. This will be done in next iteration. closes #49
@taidopurason taidopurason merged commit 3afb449 into master Nov 8, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants