Repository navigation
Deprecates "To Json" in favour of .json() method - #297
Conversation
andreagubellini
commented
Oct 9, 2020
- Reworked "To Json" test cases
- Removed pretty print test cases
- Deprecated "To Json" keyword, adding a console warning to use .json method instead
Removed sleep and replaced with 'Wait Until Http Server Is Up And Running'
|
I tried to resolve conflicts but couldn't commit to your branch even if the web interface allowed me to. |
|
I would probably keep the tests until completely removed, deprecated are still somehow supported and should be anyway tested. |
|
By the way best practice for deprecating keywords: |
|
I would like to ask you to rethink not to discontinue the KW "To JSON", I usually use it to read JSON from files (string) and turn them into a dictionary to use in KW of the Collections Library, for example, "Dictionary Should Contain Sub Dictionary". Or tell me another way to do this, because "Convert To Dictionary" from Builtin don't work for my case. |
|
I'm sorry this change has impacts on your workflow. Perhaps you could try to take a look at this library and check if it solves your problem? |
|
Thanks for the tip, I'll try! In the last case I will create a customized KW. |