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): create new file functionality on File Fields #30391

Merged
merged 17 commits into from
Oct 24, 2024

Conversation

nicobytes
Copy link
Contributor

@nicobytes nicobytes commented Oct 18, 2024

Parent Issue

#29875

Proposed Changes

This pull request introduces a new file editor component with Angular and PrimeNG, adds a store for managing the editor state, and updates related components and tests. The most important changes include the addition of the DotFormFileEditorComponent, its associated styles, constants, and store, as well as updates to the DotFormImportUrlComponent and its tests.

New File Editor Component:

Related Component Updates:

Checklist

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

Video

2024-10-23.10-30-20.mp4

This PR fixes: #29875

@nicobytes nicobytes linked an issue Oct 18, 2024 that may be closed by this pull request
@nicobytes nicobytes marked this pull request as ready for review October 23, 2024 00:27
@nicobytes nicobytes changed the title 29875 file field create new file functionality feat(edit-content): create new file functionality on File Fields Oct 23, 2024
@hmoreras hmoreras added this pull request to the merge queue Oct 24, 2024
@nicobytes nicobytes removed this pull request from the merge queue due to a manual request Oct 24, 2024
@nicobytes nicobytes added this pull request to the merge queue Oct 24, 2024
Merged via the queue into main with commit 0c1b222 Oct 24, 2024
21 checks passed
@nicobytes nicobytes deleted the 29875-file-field-create-new-file-functionality branch October 24, 2024 17:24
spbolton pushed a commit that referenced this pull request Nov 11, 2024
)

### Parent Issue

#29875 

### Proposed Changes

This pull request introduces a new file editor component with Angular
and PrimeNG, adds a store for managing the editor state, and updates
related components and tests. The most important changes include the
addition of the `DotFormFileEditorComponent`, its associated styles,
constants, and store, as well as updates to the
`DotFormImportUrlComponent` and its tests.

### New File Editor Component:

*
[`dot-form-file-editor.component.html`](diffhunk://#diff-dad0deafcdeaeb3aa5c4374c558e9e5493505545f78dcc46ec7dbaea8cff4f56R1-R59):
Added a form for file editing with fields for file name and content, and
buttons for saving or canceling the upload.
*
[`dot-form-file-editor.component.scss`](diffhunk://#diff-4508b6c98b0ef6b04cce00f33fc93d89eaf27283ec4fc731c983ad1d3e637919R1-R93):
Added styles for the file editor component, including layout and
state-specific styles.
*
[`dot-form-file-editor.component.ts`](diffhunk://#diff-4e07289537a18f138b8657c2505e3416523ee356c698aeb38f3d519a2257dadaR1-R270):
Implemented the logic for the file editor component, including form
handling, state management, and interaction with the store.
*
[`dot-form-file-editor.conts.ts`](diffhunk://#diff-32c45307bdce4649b35b4279ce966eeb847285431f1761f71cbde3caebea90f1R1-R17):
Defined default configuration options for the Monaco editor used in the
file editor component.
*
[`form-file-editor.store.ts`](diffhunk://#diff-5184979843abf3baa538afc8613a1fa040a7907c9614bfb64ddc472c31b4cfa7R1-R207):
Created a store to manage the state of the file editor, including file
upload logic and state transitions.

### Related Component Updates:

*
[`dot-form-import-url.component.html`](diffhunk://#diff-47b91a5a2108d26181aede1a70773f546e1499df45a54a630daaa7d5670f13c0L22-R22):
Updated the validation message binding to use `form.controls.url`
instead of `form.get('url')`.
*
[`dot-form-import-url.component.spec.ts`](diffhunk://#diff-0d18173fe874af517af41ad1e4422ff7681ba1c2ad4daf75c9806a07fa37e7b2R11):
Updated the test cases to use the new `ComponentStatus` enum for state
transitions.
[[1]](diffhunk://#diff-0d18173fe874af517af41ad1e4422ff7681ba1c2ad4daf75c9806a07fa37e7b2R11)
[[2]](diffhunk://#diff-0d18173fe874af517af41ad1e4422ff7681ba1c2ad4daf75c9806a07fa37e7b2L67-R68)
[[3]](diffhunk://#diff-0d18173fe874af517af41ad1e4422ff7681ba1c2ad4daf75c9806a07fa37e7b2L81-R90)

### Checklist
- [x] Tests
- [x] Translations
- [x] Security Implications Contemplated (add notes if applicable)

### Video


https://github.com/user-attachments/assets/123eb3f8-8689-4210-ae03-d31ae18688b1

---------

Co-authored-by: Arcadio Quintero <[email protected]>
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.

File field: "Create New File" functionality
3 participants