-
Notifications
You must be signed in to change notification settings - Fork 8
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
JSONPatch for off_variation doesn't work, result unchanged #17
Comments
Hi @GeosMeos thanks for creating an issue. I've identified a bug related to what you've posted here, and created a ticket to resolve it. The API should have thrown an error based on your provided In case you weren't already aware, the correct path in your snippet should be |
@tvarney13 i've assumed so as the API is pointing to offVariation but when I try to update it, an error is returned: here's the snippet:
|
@GeosMeos So in this case I was actually wrong about the path, you were right before. Sorry about that, I usually support the APIs themselves not these client libraries. I attempted a similar request on a test flag in Postman and it worked, so just to verify - you're seeing that the value for the Here's my test request (the offVariation was set to |
Hmmm, I was successfully able to update the off variation with the following:
Just to clarify, what you want to do is update the value of the flag when the flag is off? Printing the API response, this should be the value that you see updated:
|
I'm trying to patch off_variation in a specific environment(test in this case)
The following doesn't work(Note that i've taken this example from auto generated examples:
This however works with other fields such as
on
orfallthrough.variation
I've tried int and str, expected an error but operation returns 200.
Version(from pip freeze)
launchdarkly-api @ git+https://github.com/launchdarkly/api-client-python.git@9bdc16d79381a1f03c8a51dd69323c80a13a286d
The text was updated successfully, but these errors were encountered: