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

The "v" parameter with version number becomes obligatory for all API … #228

Merged
merged 1 commit into from
Mar 7, 2018

Conversation

imaleksandr
Copy link
Contributor

@imaleksandr imaleksandr commented Mar 3, 2018

The "uid" parameter obsolete. Use "id" instead.
The "v" parameter with version number becomes obligatory for all API requests. An error will be returned if the parameter is not sent.
https://vk.com/dev/version_update

…requests. An error will be returned if the parameter is not sent.
@mrbrut
Copy link

mrbrut commented Mar 5, 2018

Does anybody know when it'll be merged and released?

@TerribleDev
Copy link
Owner

@mrbrut This PR was only cut 3 days ago. I'll be looking at this ASAP, but probably by the end of the week.

@TerribleDev TerribleDev self-requested a review March 6, 2018 15:15
@TerribleDev TerribleDev added the bug label Mar 6, 2018
@TerribleDev
Copy link
Owner

@imaleksandr how is this and #227 different? I havn't yet read the docs, but I'm assuming this PR is the actual fix we want?

@TerribleDev
Copy link
Owner

oh I see, basically its not different...

This was referenced Mar 6, 2018
@TerribleDev
Copy link
Owner

@imaleksandr please see #230

@TerribleDev TerribleDev merged commit 26a6900 into TerribleDev:master Mar 7, 2018
@TerribleDev
Copy link
Owner

this will go with 2.21.0

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

Successfully merging this pull request may close these issues.

None yet

3 participants