Skip to content
This repository was archived by the owner on Jul 9, 2022. It is now read-only.

startEvent function added #686

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

darkcoderrises
Copy link

I have used some hacks to get this done. Could someone help to get it to the project's standards?

@Schmavery
Copy link
Owner

Schmavery commented Nov 24, 2018

Does this still work? I don't seem to be able to create reminders in messenger anymore.
If we want to merge it, it would be great to get a brief explanation of some of these "hacks" and then obviously we'd need docs etc.

@darkcoderrises
Copy link
Author

As you can see in the commit, I found an url, couldn't encode it properly, so just left it there as is. Maybe that's why it isn't working for you. I was hoping that you could guide me as IDK either your or fb's standards.

.post(
url1,
ctx.jar,
{}
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What happens if you pass form here instead of serializing it yourself/putting it in the url?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants