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

feat(edit-content) show files according to host/folder #30612

Conversation

nicobytes
Copy link
Contributor

@nicobytes nicobytes commented Nov 8, 2024

Parent Issue

#30216

Proposed Changes

This pull request includes several changes across multiple files to enhance the functionality and organization of the dot-site and dot-edit-content-file-field components. The key changes involve adding a new method to fetch content by folder, refactoring imports for better modularity, and updating UI components for improved user interaction.

Enhancements to dot-site.service.ts:

  • Added a new interface ContentByFolderParams to define parameters for fetching content by folder.
  • Introduced a new method getContentByFolder in DotSiteService to fetch content based on the provided parameters.

Refactoring imports in dot-edit-content-file-field components:

  • Updated import paths to use more specific model files for better modularity in various components, including dot-file-field-preview.component.ts, dot-file-field-ui-message.component.ts, dot-form-file-editor.component.ts, form-file-editor.store.ts, dot-form-import-url.component.spec.ts, and form-import-url.store.ts. [1] [2] [3] [4] [5] [6]

UI Improvements in dot-select-existing-file components:

  • Enhanced dot-dataview.component.html to include better data binding and UI elements such as loading indicators, search functionality, and improved table structure. [1] [2]
  • Updated dot-dataview.component.scss to include styles for new UI elements like truncation and thumbnail display.
  • Refactored dot-dataview.component.ts to use new Angular features like signals and models for state management.
  • Improved dot-sidebar.component.html and dot-sidebar.component.ts to handle node selection and state management more effectively. [1] [2] [3] [4] [5]
  • Updated dot-select-existing-file.component.html and dot-select-existing-file.component.ts to handle folder loading and content selection more efficiently. [1] [2] [3] [4] [5]

These changes collectively enhance the functionality, modularity, and user experience of the dot-site and dot-edit-content-file-field components.

Checklist

  • Tests
  • Translations
  • Security Implications Contemplated (add notes if applicable)

Additional Info

2024-11-08.16-57-00.mp4

This PR fixes: #30216

@nicobytes nicobytes linked an issue Nov 8, 2024 that may be closed by this pull request
@nicobytes nicobytes changed the title 30216 select existing file show files according to hostfolder with e2e feat(edit-content) show files according to host/folder Nov 8, 2024
@nicobytes nicobytes requested review from oidacra and hmoreras November 8, 2024 20:52
@nicobytes nicobytes marked this pull request as ready for review November 8, 2024 20:53
@nicobytes nicobytes requested a review from hmoreras November 8, 2024 21:49
@nicobytes nicobytes enabled auto-merge November 9, 2024 01:23
@nicobytes nicobytes disabled auto-merge November 9, 2024 01:32
@nicobytes nicobytes added this pull request to the merge queue Nov 9, 2024
Merged via the queue into main with commit 20899cc Nov 9, 2024
35 checks passed
@nicobytes nicobytes deleted the 30216-select-existing-file-show-files-according-to-hostfolder-with-e2e branch November 9, 2024 10:08
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.

Select Existing File: Show files according to Host/Folder
3 participants