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

DOC-2562: Add live-demo for document converters and full feature demo pages. #3496

Open
wants to merge 2 commits into
base: tinymce/7
Choose a base branch
from
Open

DOC-2562: Add live-demo for document converters and full feature demo pages. #3496

wants to merge 2 commits into from
+226 −29

Conversation

kemister85
Copy link
Contributor

Ticket: DOC-2562

Site: Full Feature Demo
Site: ExportWord
Site: ExportPDF
Site: ImportWord

Changes:

  • add new demos to plugin pages.

Pre-checks:

  • Branch prefixed with feature/<version>/, hotfix/<version>/, staging/<version>/, or release/<version>/.

Review:

  • Documentation Team Lead has reviewed

@kemister85 kemister85 removed the request for review from a team October 30, 2024 05:19
@kemister85 kemister85 requested a review from a team as a code owner October 30, 2024 05:19
@FarzadHayat
Copy link
Contributor

FarzadHayat commented Oct 30, 2024 •
edited by kemister85
Loading

@kemister85

  • Now that the plugins are being added to the featured featured demo, we can remove them from the excluded plugins list here:

    tinymce-docs/modules/ROOT/pages/full-featured-premium-demo.adoc

    Lines 26 to 34 in 1652b64

    |xref:exportpdf.adoc[Export to PDF]
    |Logistical concerns regarding exposing internal service URLs preclude adding this.
    |xref:exportword.adoc[Export to Word]
    |Logistical concerns regarding exposing internal service URLs preclude adding this.
    |xref:importword.adoc[Import from Word]
    |Logistical concerns regarding exposing internal service URLs preclude adding this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@FarzadHayat FarzadHayat FarzadHayat approved these changes

@lostkeys lostkeys Awaiting requested review from lostkeys

@mrinasugosh mrinasugosh Awaiting requested review from mrinasugosh

@ShiridiGandham ShiridiGandham Awaiting requested review from ShiridiGandham

@abhinavgandham abhinavgandham Awaiting requested review from abhinavgandham abhinavgandham is a code owner

@soritaheng soritaheng Awaiting requested review from soritaheng soritaheng is a code owner

@jscasca jscasca Awaiting requested review from jscasca jscasca was automatically assigned from tinymce/editor-platform-team

@MitchC1999 MitchC1999 Awaiting requested review from MitchC1999 MitchC1999 was automatically assigned from tinymce/editor-platform-team

@LewisAtTiny LewisAtTiny Awaiting requested review from LewisAtTiny LewisAtTiny is a code owner

@tiny-ben-tran tiny-ben-tran Awaiting requested review from tiny-ben-tran tiny-ben-tran was automatically assigned from tinymce/editor-platform-team

@Skylite73 Skylite73 Awaiting requested review from Skylite73 Skylite73 was automatically assigned from tinymce/editor-platform-team

At least 2 approving reviews are required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@kemister85 @FarzadHayat