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
Fix for #361, renamed username to login #362
Conversation
LGTM |
LGTM |
Any reason API is being updated (not just this PR of course, but what it is fixing) but not changed to v2? We have thousands of users/tablets with our app which logs into our Gogs/Gitea. While I understand changing "username" to "login" like Github, that breaks the API when companies like our rolled this out. Just something to remember. |
You are right, at least it should provide a fallback with a deprecation message |
@tboerger how do you implement deprecation messages for this? how do we know that |
@richmahn This was broken by a refactor in go-gogs-client a while back, I didn't really see the point in changing the api-version. it should however obviously be documented in the Changelog when 1.0 is released. 🙂 |
Also, for clients using the latest version of |
I need to search for an example we used for drone, but we have done it with a customized marshal/unmarshal |
Well sure, for |
No description provided.