Skip to content
This repository was archived by the owner on Jul 6, 2026. It is now read-only.

handle boolean settings, as {} - #604

Merged
weidongxu-microsoft merged 6 commits into
Azure:v4from
weidongxu-microsoft:v4_handle_boolean_settings
Jun 1, 2020
Merged

handle boolean settings, as {}#604
weidongxu-microsoft merged 6 commits into
Azure:v4from
weidongxu-microsoft:v4_handle_boolean_settings

Conversation

@weidongxu-microsoft

Copy link
Copy Markdown
Member

Fix #601

It is "{}" in jobject.get("result"). Not sure if there is better solution.

@jianghaolu

Copy link
Copy Markdown
Contributor

Yes! Finally!

@weidongxu-microsoft

Copy link
Copy Markdown
Member Author

There is a new test we fails. Azure/autorest.testserver#184

I will fix it in another PR.

@weidongxu-microsoft
weidongxu-microsoft merged commit df114d3 into Azure:v4 Jun 1, 2020
@weidongxu-microsoft
weidongxu-microsoft deleted the v4_handle_boolean_settings branch June 1, 2020 02:18
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants