-
-
Notifications
You must be signed in to change notification settings - Fork 413
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
[rest] Set editable for members of an item #4118
[rest] Set editable for members of an item #4118
Conversation
Required by the UI. Signed-off-by: Florian Hotze <florianh_dev@icloud.com>
@@ -317,7 +323,7 @@ public Response getItemNamespaces(@PathParam("itemname") @Parameter(description | |||
@Operation(operationId = "getItemByName", summary = "Gets a single item.", responses = { | |||
@ApiResponse(responseCode = "200", description = "OK", content = @Content(schema = @Schema(implementation = EnrichedItemDTO.class))), | |||
@ApiResponse(responseCode = "404", description = "Item not found") }) | |||
public Response getItemData(final @Context UriInfo uriInfo, final @Context HttpHeaders httpHeaders, | |||
public Response getItemByName(final @Context UriInfo uriInfo, final @Context HttpHeaders httpHeaders, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
FYI I decided to rename that method to better reflect what it is actually doing.
Signed-off-by: Florian Hotze <florianh_dev@icloud.com>
Signed-off-by: Florian Hotze <florianh_dev@icloud.com>
@kaikreuzer This PR would also be nice to have for the snapshot, as it is required to fix an UI bug. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm, thanks!
Depends on openhab/openhab-core#4118. @ghys could you comment on the purpose of `ignoreEditable` here? It's preventing the lock icon from showing up. Signed-off-by: Jimmy Tanagra <jcode@tanagra.id.au>
This sets the
editable
field of theEnrichedItemDTO
formembers
of theEnrichedGroupItemDTO
.The
editable
field is needed by the UI, e.g. for openhab/openhab-webui#2410.