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

ChatGetPermalink endpoint: change message_ts type to string #75

Closed
StyxUA opened this issue Mar 20, 2020 · 2 comments
Closed

ChatGetPermalink endpoint: change message_ts type to string #75

StyxUA opened this issue Mar 20, 2020 · 2 comments

Comments

@StyxUA
Copy link

StyxUA commented Mar 20, 2020

Similar to #12, #22, #26 and #27.

@StyxUA StyxUA changed the title ChatGetPermalink endpoint: change message_ts type to string ChatGetPermalink endpoint: change message_ts type to string Mar 20, 2020
@pyrech
Copy link
Member

pyrech commented Apr 17, 2020

It looks like there is a lot of inconsistencies in how Slack describes/provides their timestamp 😆

We would be very glad if you could open a PR to fix this endpoint 😉 We have some documentation explaining how to update the SDK: https://github.com/jolicode/slack-php-api/blob/master/doc/updating-sdk.md#generating-a-new-patch

Do not hesitate to ask questions or for some help here 😉

@damienalexandre
Copy link
Member

I think this issue is resolved already:

$optionsResolver->setAllowedTypes('message_ts', ['string']);

Feel free to reopen if you still have any problem with the message_ts.

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

No branches or pull requests

3 participants