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

[WC-2890] doc viewer: add docx support #1519

Merged
merged 8 commits into from
Apr 10, 2025
Merged

[WC-2890] doc viewer: add docx support #1519

merged 8 commits into from
Apr 10, 2025

Conversation

gjulivan
Copy link
Collaborator

@gjulivan gjulivan commented Apr 1, 2025

adding docx viewer using docx to html converter library: mammothjs

automatically detect file type

@gjulivan gjulivan requested a review from a team as a code owner April 1, 2025 22:39
@gjulivan gjulivan changed the title [WC-2890] doc viewer: add docx support [DRAFT][WC-2890] doc viewer: add docx support Apr 1, 2025
@gjulivan gjulivan changed the title [DRAFT][WC-2890] doc viewer: add docx support [WC-2890] doc viewer: add docx support Apr 1, 2025
@gjulivan gjulivan force-pushed the document_docx_viewer branch 7 times, most recently from 88251ff to 9a59c25 Compare April 9, 2025 12:11
@gjulivan gjulivan force-pushed the document_docx_viewer branch from 9a59c25 to 2acda99 Compare April 9, 2025 12:18
@gjulivan gjulivan force-pushed the document_docx_viewer branch from 2acda99 to 370ce65 Compare April 9, 2025 12:48
@gjulivan gjulivan closed this Apr 10, 2025
@gjulivan gjulivan reopened this Apr 10, 2025
@gjulivan gjulivan force-pushed the document_docx_viewer branch 5 times, most recently from 208436b to bf00e9f Compare April 10, 2025 09:16
@gjulivan gjulivan force-pushed the document_docx_viewer branch from bf00e9f to e95a9a7 Compare April 10, 2025 09:33
Copy link
Collaborator

@iobuhov iobuhov left a comment

Choose a reason for hiding this comment

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

LGTM

@gjulivan gjulivan enabled auto-merge (rebase) April 10, 2025 13:33
@gjulivan gjulivan merged commit 16fdb9d into main Apr 10, 2025
16 checks passed
@gjulivan gjulivan deleted the document_docx_viewer branch April 10, 2025 14:14
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.

3 participants