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.

Sign up for GitHub

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

Jump to bottom

feat: File Manager - ACO integration #3312

Merged
merged 39 commits into from
May 17, 2023
Merged

feat: File Manager - ACO integration #3312

merged 39 commits into from
May 17, 2023

Conversation

leopuleo
Copy link
Contributor

Changes

With this PR we add ACO to File Manager.
Screenshot 2023-04-26 at 20 42 50

Features

  • New table list view available.
  • Both Grid and Table list is generated using ACO search records.
  • Users can organise files in folders.
  • Tags rendered are coming from ACO search records.
  • Search is integrated with ACO.
  • File record is fetched only when the user opens the "File Details" drawer.
  • The file mime-type filtering is handled via tags.

How Has This Been Tested?

Jest + Cypress

Documentation

TBD

leopuleo and others added 30 commits April 26, 2023 11:49
@leopuleo
feat: add api-file-manager - aco integration (#3199)
039db50
@leopuleo
chore: add missing createdBy
9c3c8c0
@leopuleo
Merge branch 'next' into feat/file-manager-aco
ad6135c
@leopuleo
feat: app-file-manager - aco integration (#3259)
f9e60b7
@leopuleo
Merge branch 'next' into feat/file-manager-aco
d483c38
@leopuleo
fix: remove tags from search data envelope
aa8e7da
@leopuleo
chore: merge next
8314da5
@leopuleo
feat: add multipart upload
4b5a819
@leopuleo
feat: change translate name-space
de6a1c0
@leopuleo
chore: import order
79032b2
@leopuleo
style: add BottomInfoBarInner
606fc9c
@leopuleo
Merge branch 'next' into feat/file-manager-aco
985af78
@brunozoric
test(api-file-manager-aco): enable multiple cms storage operations
a3c5b61
@leopuleo
Merge branch 'next' into feat/file-manager-aco
09ed2df
@leopuleo
Merge remote-tracking branch 'origin/feat/file-manager-aco' into feat…
5c2e9ea 
…/file-manager-aco
@leopuleo
fix: remove select cta in case onChange function is not passed
c0d1e7c
@leopuleo
fix: checkbox and action column width
ab5daa4
@leopuleo
feat: add loader to File Details
dc970e9
@leopuleo
Merge branch 'next' into feat/file-manager-aco
e0e5d44
@leopuleo
Merge branch 'next' into feat/file-manager-aco
80c0614
@leopuleo
chore: fix blank page after moving record into folder
8b6ccb2
@leopuleo
Merge branch 'fix/various-file-manager-aco' into feat/file-manager-aco
0e4189e
@leopuleo
chore: fix no aco file detail
f294901
@leopuleo
fix: file details close button
54a131f
@leopuleo
chore: add props to default FileDetails
61720ea
@leopuleo
fix: skip aco search record creation for private files (#3288)
f10d15c
@leopuleo @Pavel910
feat: migrate File Manager files -> create ACO search records (#3284)
7a6ef9e 
Co-authored-by: Pavel Denisjuk <pavel@webiny.com>
@leopuleo
feat: make tags more visible
34855aa
@leopuleo
fix: table cut inside viewport
85c9b00
@leopuleo
Merge remote-tracking branch 'origin/feat/file-manager-aco' into feat…
06ad50d 
…/file-manager-aco
@leopuleo
feat: add count to listTag response
8c2bd45
@leopuleo
Merge branch 'next' into feat/file-manager-aco
f15aad6 
# Conflicts:
#	packages/api-aco/__tests__/record.so.test.ts
#	packages/api-headless-cms-ddb-es/src/operations/entry/index.ts
#	packages/api-headless-cms-ddb/src/operations/entry/index.ts
#	packages/api-headless-cms/src/graphql/schema/createManageResolvers.ts
#	packages/api-headless-cms/src/types.ts
@leopuleo
Merge branch 'next' into feat/file-manager-aco
ebe1139
@leopuleo
chore: merge conflict
ad8f628
@leopuleo
chore: various fixes
d14e078
@leopuleo
chore: run lint
bbd3e72
@leopuleo
fix: enforce security rules for listTags
3d37326
@leopuleo
fix: loading on file click
1fb65cc
@leopuleo leopuleo self-assigned this May 17, 2023
@leopuleo leopuleo added this to the 5.36.0 milestone May 17, 2023
@leopuleo leopuleo marked this pull request as ready for review May 17, 2023 08:13
Pavel910
Pavel910 approved these changes May 17, 2023
View reviewed changes
Copy link
Collaborator

@Pavel910 Pavel910 left a comment

Choose a reason for hiding this comment

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

Let's squash merge this!

@Pavel910 Pavel910 merged commit 3b0b561 into next May 17, 2023
@leopuleo leopuleo deleted the feat/file-manager-aco branch June 5, 2023 07:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Pavel910 Pavel910 Pavel910 approved these changes

@brunozoric brunozoric Awaiting requested review from brunozoric

Assignees

@leopuleo leopuleo

Labels
None yet
Projects
None yet
Milestone
5.36.0
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@leopuleo @Pavel910 @brunozoric