Skip to content
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

Merged
merged 17 commits into from
Aug 15, 2022
Merged

[full-ci] Tags #7385

merged 17 commits into from
Aug 15, 2022

Conversation

AlexAndBear
Copy link
Contributor

@AlexAndBear AlexAndBear commented Aug 2, 2022

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

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Technical debt
  • Tests

Checklist:

  • Code changes
  • Unit tests added
  • Acceptance tests added
  • Documentation ticket raised

Open tasks:

  • CI

@update-docs
Copy link

update-docs bot commented Aug 2, 2022

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.

@AlexAndBear AlexAndBear requested review from kulmann and fschade August 2, 2022 10:54
@ownclouders
Copy link
Contributor

ownclouders commented Aug 2, 2022

Results for oCISFiles2 https://drone.owncloud.com/owncloud/web/27539/57/1

💥 The acceptance tests failed on retry. Please find the screenshots inside ...

webUIFilesActionMenu-fileFolderActionMenu_feature-L14.png

webUIFilesActionMenu-fileFolderActionMenu_feature-L14.png

webUIFilesActionMenu-fileFolderActionMenu_feature-L37.png

webUIFilesActionMenu-fileFolderActionMenu_feature-L37.png

@fschade fschade force-pushed the tags-experimental branch from 416e779 to 875e6df Compare August 11, 2022 15:56
@fschade fschade marked this pull request as ready for review August 11, 2022 15:56
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
@fschade fschade force-pushed the tags-experimental branch from 875e6df to e26bc50 Compare August 11, 2022 16:00
@AlexAndBear AlexAndBear changed the title Tags init Tags Aug 11, 2022
@fschade fschade changed the title Tags [full-ci] Tags Aug 12, 2022
Copy link
Contributor

@fschade fschade left a 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]
Copy link
Contributor

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.

Copy link
Contributor

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.

Copy link
Contributor Author

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 ;)

@sonarqubecloud
Copy link

SonarCloud Quality Gate failed.    Quality Gate failed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 1 Code Smell

35.6% 35.6% Coverage
0.0% 0.0% Duplication

@AlexAndBear AlexAndBear merged commit 5eabab0 into experimental Aug 15, 2022
@delete-merged-branch delete-merged-branch bot deleted the tags-experimental branch August 15, 2022 13:48
fschade added a commit that referenced this pull request Aug 16, 2022
* 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]>
JammingBen pushed a commit that referenced this pull request Dec 5, 2022
JammingBen pushed a commit that referenced this pull request Dec 8, 2022
JammingBen pushed a commit that referenced this pull request Dec 9, 2022
fschade pushed a commit that referenced this pull request Dec 20, 2022
fschade pushed a commit that referenced this pull request Dec 21, 2022
fschade pushed a commit that referenced this pull request Dec 22, 2022
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.

4 participants