Skip to content

feat: resources can be tagged#970

Merged
mburri merged 11 commits intomasterfrom
feat/tag-resources
Mar 25, 2026
Merged

feat: resources can be tagged#970
mburri merged 11 commits intomasterfrom
feat/tag-resources

Conversation

@mburri
Copy link
Copy Markdown
Member

@mburri mburri commented Mar 6, 2026

Adds "more" -> "tag current state" menu item and functionality

image

@mburri mburri requested a review from StephGit March 6, 2026 12:52
Base automatically changed from development to master March 13, 2026 18:42
@mburri mburri changed the title Feat/tag resources feat: resources can be tagged Mar 20, 2026
@mburri mburri marked this pull request as ready for review March 20, 2026 08:25
Comment thread AMW_angular/io/src/app/resources/resource-edit/resource-edit.component.html Outdated
Comment thread AMW_angular/io/src/app/resources/services/resource-tags.service.ts Outdated
Comment thread AMW_angular/io/src/app/resources/resource-edit/resource-edit.component.ts Outdated
return Response.status(NOT_FOUND).entity(new ExceptionDto("Resource not found")).build();
}

List<ResourceTagEntity> tags = tagConfigurationService.loadTagLabelsForResource(resource);
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No permission check for READ. Add @HasPermission in Service

Comment thread AMW_rest/src/main/java/ch/mobi/itc/mobiliar/rest/resources/ResourceTagsRest.java Outdated
Comment thread AMW_rest/src/main/java/ch/mobi/itc/mobiliar/rest/resources/ResourceTagsRest.java Outdated
Comment thread AMW_rest/src/main/java/ch/mobi/itc/mobiliar/rest/resources/ResourceTagsRest.java Outdated
@mburri mburri force-pushed the feat/tag-resources branch from f2c3fba to 373ac0f Compare March 25, 2026 15:19
@mburri mburri merged commit a242706 into master Mar 25, 2026
1 check passed
@yvespp yvespp added this to the Release v1.18.11 milestone Apr 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants