-
Notifications
You must be signed in to change notification settings - Fork 155
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
[full-ci] Tags #7385
[full-ci] Tags #7385
Conversation
Thanks for opening this pull request! The maintainers of this repository would appreciate it if you would create a changelog item based on your changes. |
Results for oCISFiles2 https://drone.owncloud.com/owncloud/web/27539/57/1 |
416e779
to
875e6df
Compare
add keys tags key to filter search bump sdk to experimental version open details sidebar instead of tags sidebar panel when navigation through files table more tags link update search input value when search query changes add list all tags endpoint and use it when entering a tag phrase
875e6df
to
e26bc50
Compare
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.
just some optional nitpicks, please decide if those make sense or not.
} | ||
}, | ||
mounted() { | ||
this.editAssignedTags = [...this.resource.tags] |
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.
why unspreading? the contained objects are still references.
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.
happens multiple times in the component.
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.
Containing objects are just strings, and they won't be edited...
We need to make sure, that the revert action still works -> which it does ;)
SonarCloud Quality Gate failed. |
* change ocis to experimental change sdk to experimental * bump ocis commit id * Add tags to ResourceTable (#7388) * Add tags to resourcetable * Use file tags instead of tags * Add changelog, remove border for + item * Replace OcFileTag * Dump ODS * [full-ci] Tags (#7385) Tags init * Truncate tags (#7442) Truncate tags Co-authored-by: Paul Neubauer <[email protected]> Co-authored-by: Jan <[email protected]>
Description
EXPERIMENTAL!
Managing files via tags is now possible in web, the feature is experimental and will be only available through a dedicated experimental web build.
Beside that the web version is experimental, it also needs a special experimental ocis version.
Creating Tags, tagging resources and search for tags now is possible and can be used as an alternative way of working and organizing resources.
Types of changes
Checklist:
Open tasks: