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

feat: create Amazon bundle #7255

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

feat: create Amazon bundle #7255

wants to merge 5 commits into from

Conversation

edwinjosechittilappilly
Copy link
Collaborator

Screenshot 2025-03-25 at 3 52 29 AM This pull request includes several changes to the organization and import paths of Amazon-related components in the `langflow` project. The most important changes involve moving the Amazon components to a new directory and updating the import statements accordingly.

Reorganization of Amazon components:

Removal of old Amazon component imports:

Test updates:

Frontend updates:

@dosubot dosubot bot added the size:S This PR changes 10-29 lines, ignoring generated files. label Mar 25, 2025
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR reorganizes Amazon-related components within the langflow project to centralize them under a new directory structure and update the corresponding import paths accordingly.

  • Moves Amazon components into the src/backend/base/langflow/components/amazon directory
  • Updates import references across models, embeddings, data, and tests
  • Updates frontend display by adding an entry for Amazon in the SIDEBAR_BUNDLES array

Reviewed Changes

Copilot reviewed 10 out of 10 changed files in this pull request and generated no comments.

Show a summary per file
File Description
src/backend/base/langflow/components/amazon/init.py New file to consolidate Amazon-related component imports
src/frontend/src/utils/styleUtils.ts Adds Amazon to the sidebar bundles for frontend display
src/backend/base/langflow/components/models/init.py Removes legacy AmazonBedrockComponent import and all update
src/backend/base/langflow/components/data/init.py Removes S3BucketUploaderComponent from imports and all
src/backend/base/langflow/components/embeddings/init.py Removes AmazonBedrockEmbeddingsComponent from imports and all
src/backend/base/langflow/base/models/model_input_constants.py Updates import path for AmazonBedrockComponent
src/backend/tests/unit/components/data/test_s3_uploader_component.py Updates import path for S3BucketUploaderComponent

@github-actions github-actions bot added the enhancement New feature or request label Mar 25, 2025
Copy link

codspeed-hq bot commented Mar 25, 2025

CodSpeed Performance Report

Merging #7255 will improve performances by 22.87%

Comparing awsbundle (f6f8911) with main (1218246)

Summary

⚡ 1 improvements
✅ 18 untouched benchmarks

Benchmarks breakdown

Benchmark BASE HEAD Change
test_cancel_nonexistent_build 12 ms 9.7 ms +22.87%

Copy link
Member

@Cristhianzl Cristhianzl left a comment

Choose a reason for hiding this comment

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

can we add aws icon in s3 bucket component?

@github-actions github-actions bot added enhancement New feature or request and removed enhancement New feature or request labels Mar 25, 2025
@github-actions github-actions bot added enhancement New feature or request and removed enhancement New feature or request labels Mar 25, 2025
@github-actions github-actions bot added enhancement New feature or request and removed enhancement New feature or request labels Mar 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request size:S This PR changes 10-29 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants