-
Notifications
You must be signed in to change notification settings - Fork 56
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
Add refresh_token to OauthV2Access.php #172
Comments
Hello! Thanks for reporting this issue. It looks like it's missing from Slack OpenApi definition: That's why it's not allowed in OauthV2Access. The only way to fix this is to create a new patch: https://github.com/jolicode/slack-php-api/blob/31a49b5f54049a1ebb001de2545aff0500d4fdd6/docs/4-updating-sdk.md#generating-a-new-patch (because there specification is really not good, we keep a list of patches to improve it before building the PHP client). Would you be available to give it a try? Thanks |
Okay, I'll look into it. Will it be quickly updated after that? Because I need it for a project. |
Yes we can release a new version as soon as it's done 👍 |
I can't push my branch to make a pull request. Sorry, first time contributing to a project, I discover ^^ |
Welcome to the open-source world 🎉 To be able to contribute, you have to work on a "fork": https://github.com/jolicode/slack-php-api/fork It's this fork you must work on (create branches, make changes, etc). Then when ready, Github will display a button to open a Pull Request on the main project. |
Thank you very much. All done! |
As far as I can see, there is no possibility of using the oauthV2Access method for token refresh.
Can you add it?
The text was updated successfully, but these errors were encountered: