[NETCONF-214] API Explorer Model Schema Data Type 'Click to set as parameter value' misses to put wrapping structure? Created: 02/Jun/16 Updated: 15/Mar/19 Resolved: 22/Sep/16 |
|
| Status: | Resolved |
| Project: | netconf |
| Component/s: | restconf-nb |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | Bug | ||
| Reporter: | Michael Vorburger | Assignee: | Miroslav Kovac |
| Resolution: | Done | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Environment: |
Operating System: All |
||
| Issue Links: |
|
||||||||
| External issue ID: | 5997 | ||||||||
| Description |
|
If you try to use it (apidoc, Swagger) to e.g. PUT /config/toaster:toaster/ then a dumb end-user like me will naturally find and use that nice "example" parameter support that UI has - on e.g. the (config)toaster parameter, one can click on Model Schema under Data Type where it says e.g. { "darknessFactor": "integer" }& 'Click to set as parameter value'. Doing that will fill in { "darknessFactor": "integer" }, make that { "darknessFactor": "500" }and click that "Try it out!" button and: "Error parsing input: Schema node with name darknessFactor wasn't found..." - because it would actually have to be: {"toaster": {"darknessFactor": 500}}, only then it works. |
| Comments |
| Comment by Miroslav Kovac [ 09/Sep/16 ] |
|
Thank you for precise explanation. I made some changes and tested it. It looks fine but please take a look at this patch and test it if you can. |
| Comment by Michael Vorburger [ 09/Sep/16 ] |
|
> please take a look at this patch and test it if you can. Are you talking to me? |
| Comment by Miroslav Kovac [ 09/Sep/16 ] |
|
Sorry got distracted while writing the reply. Here you go: https://git.opendaylight.org/gerrit/#/c/45419/ |
| Comment by Michael Vorburger [ 14/Sep/16 ] |
|
FTR: The c/45419 was abandoned with "Wrong solution. Starting all over." - if you would ever like me to review another one, please post it here. |
| Comment by Miroslav Kovac [ 14/Sep/16 ] |
|
Still working on it. Seems to be more tricky than I thought. |
| Comment by Miroslav Kovac [ 19/Sep/16 ] |
|
Hi, please check out this patch https://git.opendaylight.org/gerrit/#/c/45838/. Seems like it is finally working. I ll make tests in separate patch for both bugs (5997, 5998) |