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

Merge pdfView Branch Features #1772

Merged
merged 6 commits into from
Apr 24, 2024
Merged

Merge pdfView Branch Features #1772

merged 6 commits into from
Apr 24, 2024

Conversation

Cristhianzl
Copy link
Collaborator

This pull request integrates the latest enhancements from the pdfView branch into the development (dev) branch. The pdfView branch introduces crucial features aimed at enhancing the manipulation of new input/output flows within our system. These features have been thoroughly developed and tested to ensure seamless integration and optimal functionality. By merging these changes into the dev branch, we aim to streamline our development process and provide users with enhanced capabilities for managing and manipulating PDF files, CSV or images within our application.

…o include 'separator' variable to re-render component when separator changes
…ng in file variable assignment to prevent app crash

✨ (csvOutputComponent/index.tsx): add error message and UI when file variable is null to inform user about the error
…input and return the input value as is

📝 (CSVOutput.py): Add CSVOutput component to handle CSV output with configurable separator
📝 (ImageOutput.py): Add ImageOutput component to handle image output
📝 (PDFOutput.py): Add PDFOutput component to handle PDF output
📝 (constants.ts): Add KeyPairInput to the list of supported input types
📝 (IOModal/index.tsx): Remove console.log statement
🔧 (CSVOutput.py): Remove unused CSVOutput component
🔧 (ImageOutput.py): Remove unused ImageOutput component
🔧 (PDFOutput.py): Remove unused PDFOutput component

These components were deleted as they were no longer being used in the project and were causing unnecessary clutter in the codebase. Removing them improves code maintainability and reduces potential confusion for developers working on the project.
@Cristhianzl Cristhianzl self-assigned this Apr 24, 2024
Copy link

vercel bot commented Apr 24, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
langflow ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 24, 2024 0:53am

…ield variable to clean up code and improve performance
@anovazzi1 anovazzi1 self-requested a review April 24, 2024 00:52
Copy link
Contributor

@anovazzi1 anovazzi1 left a comment

Choose a reason for hiding this comment

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

tested locally, LGTM

@anovazzi1 anovazzi1 merged commit ce322f1 into dev Apr 24, 2024
5 checks passed
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.

2 participants