-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
Field names deprecation in 0.4.0 #189
Comments
Are you sure that this is the version that's currently running. You may have a stale process running a previous version. If this isn't the case, please provide the endpoint(s) with the expected output and the actual output. Cheers. |
Hi @jvshahid – The line with the version in the previous message was pasted from the terminal after I ran So now I did the following:
Now I have a nice and empty database. I run my tests, which create a bunch of cluster admins, and when I list cluster admins, I expect to see
but what I get is
My tests also create a bunch of random databases and users, so when I list users from one of them (
but instead I get
My tests shouldn't even pass after these changes, since I'm explicitly reading from a field named |
Thanks @msrdic for your detailed explanation. It seems like we missed those two endpoints. They will get fixed in the next release. |
Hi –
I read in 0.4.0 release notes that the
username
field for/cluster_admins
and/db/:db/users
calls is deprecated in favour of a more consistentname
field. When I changed my library to work with these changes, tests failed. I discovered that I still get theusername
field, for both API calls. I have the latest brew package:InfluxDB v0.4.0 (git: 012efd4)
Am I missing something, or is the deprecation still in preparation phase?
The text was updated successfully, but these errors were encountered: