Skip to content

🔧 fix: Dev Deployment, Mistral OCR Error, and UI Consistency #7668

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

Merged
merged 9 commits into from
Jun 1, 2025

Conversation

danny-avila
Copy link
Owner

@danny-avila danny-avila commented Jun 1, 2025

Summary

Closes #7667
Closes #7603

I updated the deployment workflow conditions, fixed a Mistral OCR bug affecting document files, and improved several UI components for better user experience and maintainability.

  • Updated deploy workflow to include conditions for successful dev branch deployment and streamlined deployment steps
  • Fixed .docx/.pptx Mistral OCR error by adding image_limit=0 parameter to prevent processing failures
  • Simplified placeholder logic in VariableForm component to use variable name directly instead of localized template
  • Enhanced icon rendering in TTS components by consolidating logic and improving button rendering consistency
  • Updated ProgressText and ToolCall components for improved layout consistency with better flex properties
  • Refactored ProgressText component logic for better readability and maintainability with cleaner conditional rendering
  • Formatted ESLint configuration for improved readability and removed unused nested ternary rule
  • Improved error handling in ProgressText and ToolCall components with proper localization for cancelled states

Change Type

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)

Testing

Test Configuration:

  • Tested Mistral OCR with .docx and .pptx files to ensure the image_limit parameter resolves processing errors
  • Verified TTS component functionality in both browser and external modes
  • Confirmed ProgressText component properly displays error, loading, and completed states
  • Validated deployment workflow changes don't break existing CI/CD processes

Checklist

  • My code adheres to this project's style guidelines
  • I have performed a self-review of my own code
  • I have commented in any complex areas of my code
  • I have made pertinent documentation changes
  • My changes do not introduce new warnings
  • Local unit tests pass with my changes

@danny-avila danny-avila merged commit f9d4078 into dev Jun 1, 2025
7 checks passed
@danny-avila danny-avila deleted the fix/tool-call-ui branch June 1, 2025 21:48
arthurolivierfortin pushed a commit to arthurolivierfortin/LibreChat-Fork that referenced this pull request Jun 4, 2025
…vila#7668)

* 🔧 fix: Update ProgressText and ToolCall components for improved error handling and localization

* 🔧 chore: Format ESLint configuration for improved readability and remove unused rule

* 🔧 refactor: Simplify ProgressText component logic for better readability and maintainability

* 🔧 refactor: Update ProgressText and ToolCall components for improved layout consistency

* 🔧 refactor: Simplify icon rendering in TTS components and enhance button rendering logic in HoverButtons

* 🔧 refactor: Update placeholder logic in VariableForm component to simply use variable name

* fix: .docx. .pptx Mistral OCR Error with `image_limit=0`

* chore: Update deploy workflow to include conditions for successful dev branch deployment and streamline deployment steps

* ci: Set image_limit to 0 in MistralOCR service tests for consistent behavior
kenshinsamue pushed a commit to intelequia/LibreChat that referenced this pull request Aug 4, 2025
…vila#7668)

* 🔧 fix: Update ProgressText and ToolCall components for improved error handling and localization

* 🔧 chore: Format ESLint configuration for improved readability and remove unused rule

* 🔧 refactor: Simplify ProgressText component logic for better readability and maintainability

* 🔧 refactor: Update ProgressText and ToolCall components for improved layout consistency

* 🔧 refactor: Simplify icon rendering in TTS components and enhance button rendering logic in HoverButtons

* 🔧 refactor: Update placeholder logic in VariableForm component to simply use variable name

* fix: .docx. .pptx Mistral OCR Error with `image_limit=0`

* chore: Update deploy workflow to include conditions for successful dev branch deployment and streamline deployment steps

* ci: Set image_limit to 0 in MistralOCR service tests for consistent behavior
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