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

Edit Contentlet: Allow File Preview #26613

Merged

Conversation

rjvelazco
Copy link
Contributor

@rjvelazco rjvelazco commented Nov 7, 2023

Description

🤖 Generated by Copilot at e287a83

Improved the binary field component and its storybook and test files. Removed unused dependencies and operators from the binary-field.component.stories.ts file. Added logic and tests for previewing and editing unknown file types in the dot-binary-field-preview component.

Related Issue

Fixes #26105

Explanation of Changes

🤖 Generated by Copilot at e287a83

  • Added unknown value to EDITABLE_FILE enum and updated logic for setting contenttype and isEditable properties to handle unknown files in the preview component (link,link,link)
  • Removed unused imports and providers from the storybook component and fixed flaky tests (link,link)
  • Added constant fileUnknown and test suite for unknown files in the preview component spec file (link,link)

Video

Screen.Recording.2023-11-07.at.11.22.43.AM.mov

@rjvelazco rjvelazco requested review from fmontes and zJaaal and removed request for fmontes November 7, 2023 15:23
Copy link

Copy link

SonarQube Quality Gate

Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@fmontes fmontes merged commit 7667284 into master Nov 7, 2023
28 of 33 checks passed
@fmontes fmontes deleted the issue-26105-edit-contentlet-allow-file-preview-fix-edit-code branch November 7, 2023 17:34
@fmontes
Copy link
Member

fmontes commented Nov 7, 2023

Pipeline is failing, but since this code is not related to Java is ok to merge.

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.

Edit Contentlet: Allow File Preview
4 participants