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

Implement support for nested buckets view #146

Merged
merged 5 commits into from
Nov 20, 2023
Merged

Conversation

kyle1morel
Copy link
Contributor

@kyle1morel kyle1morel commented Nov 14, 2023

Description

Changes the bucket list view to use a TreeTable instead of a DataTable. This allows us to mimic a folder hierarchy of buckets based on their canonical path (endpoint, bucket, key). Fake folders will be inserted to create a full tree if there is no direct parent bucket available.

This PR also slipstream resolves a build issue with the Dockerfile.

Types of changes

Bug fix (non-breaking change which fixes an issue)
New feature (non-breaking change which adds functionality)

Checklist

  • I have read the CONTRIBUTING doc
  • I have checked that unit tests pass locally with my changes
  • I have added tests that prove my fix is effective or that my feature works
  • I have added necessary documentation (if appropriate)

Further comments

Copy link

Coverage Report (Application)

Totals Coverage
Statements: 70.67% ( 53 / 75 )
Methods: 62.5% ( 5 / 8 )
Lines: 81.63% ( 40 / 49 )
Branches: 44.44% ( 8 / 18 )

Copy link

github-actions bot commented Nov 14, 2023

Coverage Report (Frontend)

Totals Coverage
Statements: 34.31% ( 630 / 1836 )
Methods: 33.33% ( 133 / 399 )
Lines: 41.07% ( 444 / 1081 )
Branches: 14.89% ( 53 / 356 )

@kyle1morel kyle1morel force-pushed the feature/treetable branch 5 times, most recently from 015e360 to 331e33c Compare November 15, 2023 18:17
@kyle1morel kyle1morel changed the title Convert bucket list to a TreeTable Nested buckets view Nov 15, 2023
@kyle1morel kyle1morel marked this pull request as ready for review November 15, 2023 18:34
@kyle1morel kyle1morel changed the title Nested buckets view Implement support for nested buckets view Nov 17, 2023
frontend/src/components/bucket/BucketTable.vue Outdated Show resolved Hide resolved
frontend/src/components/bucket/BucketTable.vue Outdated Show resolved Hide resolved
frontend/src/components/bucket/BucketTable.vue Outdated Show resolved Hide resolved
frontend/src/components/bucket/BucketTable.vue Outdated Show resolved Hide resolved
frontend/src/components/bucket/BucketTable.vue Outdated Show resolved Hide resolved
frontend/src/components/bucket/BucketTable.vue Outdated Show resolved Hide resolved
frontend/src/types/BucketTreeNode.ts Outdated Show resolved Hide resolved
frontend/src/types/BucketTreeNode.ts Outdated Show resolved Hide resolved
frontend/src/utils/utils.ts Outdated Show resolved Hide resolved
@kyle1morel kyle1morel force-pushed the feature/treetable branch 2 times, most recently from bbe390c to c45cd10 Compare November 17, 2023 21:57
@norrisng-bc norrisng-bc merged commit 33b6b7a into master Nov 20, 2023
18 checks passed
@norrisng-bc norrisng-bc deleted the feature/treetable branch November 20, 2023 20:34
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