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

Webhook messaging listener can't access the timestamp field #1097

Closed
alex-lx opened this issue Sep 25, 2020 · 1 comment
Closed

Webhook messaging listener can't access the timestamp field #1097

alex-lx opened this issue Sep 25, 2020 · 1 comment
Assignees
Milestone

Comments

@alex-lx
Copy link

alex-lx commented Sep 25, 2020

when implementing com.restfb.webhook.WebhookMessagingListener#message, there's no way to access the com.restfb.types.webhook.messaging.MessagingItem#timestamp.

@nbartels
Copy link
Contributor

We modified the listener to have the method duplicated with an additional timestamp parameter. The old methods are marked deprecated. For now we added all the new methods to our abstract class. And so it should be possible very easily to migrate to the new methods.

In the next version we go with this method duplication and remove it in the later releases. So everyone has the possibility to adapt the code before the old methods are removed.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Dec 28, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants