Skip to content

Document PowerPoint add-in production deployment issue - #79

Open
iliasbet wants to merge 1 commit into
mainfrom
fix/document-powerpoint-production-issue
Open

Document PowerPoint add-in production deployment issue#79
iliasbet wants to merge 1 commit into
mainfrom
fix/document-powerpoint-production-issue

Conversation

@iliasbet

@iliasbet iliasbet commented Oct 9, 2025

Copy link
Copy Markdown

Issue

User reported (Oct 9, 2025) that the PowerPoint add-in shows InvalidArgument error when processing slides.

Root Cause

Production code at office-addins.dust.tt is outdated and doesn't include the fixes from September 11, 2025.

The problem: Production tries to read text from all shapes (including images/charts), which don't have text properties.

The fix: Code was updated to filter shapes by type first, only reading text from text-capable shapes. This fix is already in the repository but not deployed.

What This PR Does

Adds documentation explaining:

  • What the user issue is
  • Why production is broken
  • That the fix already exists in the repo
  • Next step: deploy current main branch to production

Related Commits

The bug fix commits (Sept 11, 2025):

  • f7705d7 - safeExtractTextFromShapes()
  • c7c1846 - Only use text capable shapes
  • d50570c - Don't break on non-text shapes
  • a8c0a62 - Cascading error handling
  • 3f563c2 - Fix shape try catch

Action Needed

Deploy the current main branch to production:

cd office && vercel --prod

Testing

After deployment, verify with PowerPoint containing text + images + charts.

From user ticket (Oct 9, 2025): Add-in shows InvalidArgument error

- Bug was fixed in September 2025 (commits f7705d7-be6fdc1)
- Fix is in repository but not deployed to production
- Production needs deployment to resolve user issue
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.

1 participant