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

Allow a person to tag buckets. #419

Merged
merged 1 commit into from
Dec 8, 2021
Merged

Allow a person to tag buckets. #419

merged 1 commit into from
Dec 8, 2021

Conversation

csc-felipe
Copy link
Contributor

@csc-felipe csc-felipe commented Nov 30, 2021

Description

Gives a person the possibility to add or remove tags from a bucket. Includes the ability to show or hide tags when browsing the buckets.

Please, check the Finnish language :)

Related issues

Related to #85.

Type of change

  • New feature (non-breaking change which adds functionality)

Changes Made

Added backend for updating metadata for a bucket. Added UI for displaying tags for containers, as well as hiding them. There is a new button, called 'Edit', which makes it possible to edit a bucket's tags. When creating a new Bucket, it's possible to add tags to it.

Tags are not displayed in the bucket view (after double click on a bucket). There's no edit button on that page either.

Testing

  • Unit Tests
  • Integration Tests

Mentions

Thanks to @sampsapenna for solving my javascript sync/async mess.

@csc-felipe
Copy link
Contributor Author

cypress is clicking the wrong switch
Browse buckets and test operations -- should be able to filter table, adjust display buckets per page and pagination (failed)
.

@blankdots
Copy link
Contributor

blankdots commented Dec 1, 2021

cypress is clicking the wrong switch

then we need a better selector for that switch, or something is off with clicking the parent cy.contains('Paginated').parent().click() because you put them in the same control div, moving them to separate divs should help
or
finding out a new selector

@blankdots blankdots linked an issue Dec 2, 2021 that may be closed by this pull request
@csc-felipe csc-felipe force-pushed the feature/usertags branch 2 times, most recently from 7687ed5 to fa6bdea Compare December 3, 2021 06:41
@csc-felipe csc-felipe changed the title WIP: Allow a person to tag buckets. Allow a person to tag buckets. Dec 3, 2021
Copy link
Member

@sampsapenna sampsapenna left a comment

Choose a reason for hiding this comment

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

Some small alterations to the Finnish upon further inspection, other than that LGTM.

Copy link
Contributor

@blankdots blankdots left a comment

Choose a reason for hiding this comment

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

the new POST endpoint "/api/bucket/meta" should be added to the API.yaml

swift_browser_ui_frontend/src/views/AddContainer.vue Outdated Show resolved Hide resolved
@blankdots blankdots added the enhancement New feature or request label Dec 3, 2021
This feature has a frontend and backend part. From the frontend, user can create tags at the create bucket screen. On the backend, the tags are stored as metadata in the swift object store.
@csc-felipe csc-felipe dismissed stale reviews from blankdots and sampsapenna December 7, 2021 09:58

Added entry to API docs

@csc-felipe csc-felipe merged commit a28efa3 into devel Dec 8, 2021
@csc-felipe csc-felipe deleted the feature/usertags branch December 8, 2021 07:17
csc-felipe added a commit that referenced this pull request Dec 9, 2021
This is a continuation to the tagging buckets feature from PR #419.
The openApi spec supports tuples in 3.1 with prefixItems. We may want to change the api docs when that comes into use.
@csc-felipe csc-felipe mentioned this pull request Dec 9, 2021
3 tasks
csc-felipe added a commit that referenced this pull request Dec 9, 2021
This is a continuation to the tagging buckets feature from PR #419.
The openApi spec supports tuples in 3.1 with prefixItems. We may want to change the api docs when that comes into use.
csc-felipe added a commit that referenced this pull request Dec 10, 2021
This is a continuation to the tagging buckets feature from PR #419.
The openApi spec supports tuples in 3.1 with prefixItems. We may want to change the api docs when that comes into use.
csc-felipe added a commit that referenced this pull request Dec 10, 2021
This is a continuation to the tagging buckets feature from PR #419.
The openApi spec supports tuples in 3.1 with prefixItems. We may want to change the api docs when that comes into use.
csc-felipe added a commit that referenced this pull request Dec 10, 2021
This is a continuation to the tagging buckets feature from PR #419.
The openApi spec supports tuples in 3.1 with prefixItems. We may want to change the api docs when that comes into use.
csc-felipe added a commit that referenced this pull request Dec 10, 2021
csc-felipe added a commit that referenced this pull request Dec 10, 2021
This is a continuation to the tagging buckets feature from PR #419.
The openApi spec supports tuples in 3.1 with prefixItems. We may want to change the api docs when that comes into use.
csc-felipe added a commit that referenced this pull request Dec 10, 2021
This is a continuation to the tagging buckets feature from PR #419.
The openApi spec supports tuples in 3.1 with prefixItems. We may want to change the api docs when that comes into use.
@csc-felipe csc-felipe mentioned this pull request Dec 10, 2021
2 tasks
csc-felipe added a commit that referenced this pull request Dec 10, 2021
This is a continuation to the tagging buckets feature from PR #419.
The openApi spec supports tuples in 3.1 with prefixItems. We may want to change the api docs when that comes into use.
csc-felipe added a commit that referenced this pull request Dec 15, 2021
This is a continuation to the tagging buckets feature from PR #419.
The openApi spec supports tuples in 3.1 with prefixItems. We may want to change the api docs when that comes into use.
csc-felipe added a commit that referenced this pull request Dec 15, 2021
csc-felipe added a commit that referenced this pull request Dec 15, 2021
This is a continuation to the tagging buckets feature from PR #419.
The openApi spec supports tuples in 3.1 with prefixItems. We may want to change the api docs when that comes into use.
csc-felipe added a commit that referenced this pull request Dec 15, 2021
csc-felipe added a commit that referenced this pull request Dec 20, 2021
This is a continuation to the tagging buckets feature from PR #419.
The openApi spec supports tuples in 3.1 with prefixItems. We may want to change the api docs when that comes into use.
csc-felipe added a commit that referenced this pull request Dec 20, 2021
csc-felipe added a commit that referenced this pull request Dec 20, 2021
@sampsapenna sampsapenna mentioned this pull request Jan 12, 2022
8 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Using metadata to organize container files
3 participants