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: S3バケット名の変更と新規バケット追加 #260

Merged
merged 4 commits into from
Jan 1, 2025

Conversation

mt-osho-san
Copy link
Collaborator

@mt-osho-san mt-osho-san commented Dec 29, 2024

User description

概要

S3 バケットの名前を修正しました。

変更点

original_image_bucket : 花瓶、ラッピング、花の元画像
flower_bucket : ユーザーごとの花瓶、ラッピングの画像
bouquet_bucket : ユーザーごとの花束の画像

影響範囲

Bucket を利用している箇所の変数を変えたが、機能としての変更はないです。

テスト

とくになし。

関連 Issue


PR Type

Enhancement


Description

  • S3バケットの名称をflowerImageBucketからoriginalImageBucketに変更しました。
  • 新たにflowerBucketを追加しました。
  • これに伴い、関連する環境変数名も変更されています。

Changes walkthrough 📝

Relevant files
Configuration changes
7 files
backend-stack.ts
S3バケット名の変更と新規バケット追加                                                                           

src/backend/lib/backend-stack.ts

  • flowerImageBucketoriginalImageBucketに変更
  • flowerBucketを新たに追加
  • +3/-2     
    bouquet.ts
    バケット名の変更                                                                                                 

    src/backend/lib/constructs/bouquet.ts

    • flowerImageBucketoriginalImageBucketに変更
    +3/-3     
    diary.ts
    バケット名の変更と新規バケット追加                                                                               

    src/backend/lib/constructs/diary.ts

  • flowerImageBucketoriginalImageBucketに変更
  • flowerBucketを新たに追加
  • +6/-3     
    flower.ts
    バケット名の変更と新規バケット追加                                                                               

    src/backend/lib/constructs/flower.ts

  • flowerImageBucketoriginalImageBucketに変更
  • flowerBucketを新たに追加
  • +19/-8   
    bouquet_create.py
    環境変数名の変更                                                                                                 

    src/backend/lambda/bouquet_create/bouquet_create.py

    • 環境変数FLOWER_BUCKET_NAMEORIGINAL_IMAGE_BUCKET_NAMEに変更
    +2/-2     
    diary_create.py
    環境変数名の変更                                                                                                 

    src/backend/lambda/diary_create/diary_create.py

    • 環境変数FLOWER_IMAGE_BUCKET_NAMEORIGINAL_IMAGE_BUCKET_NAMEに変更
    +1/-1     
    flower_get.py
    環境変数名の変更                                                                                                 

    src/backend/lambda/flower_get/flower_get.py

    • 環境変数BUCKET_NAMEFLOWER_BUCKET_NAMEに変更
    +1/-1     
    Tests
    2 files
    test_diary_create.py
    テスト用環境変数名の変更                                                                                         

    src/backend/test/pytest/test_diary_create.py

    • 環境変数FLOWER_IMAGE_BUCKET_NAMEORIGINAL_IMAGE_BUCKET_NAMEに変更
    +2/-2     
    test_flower_get.py
    テスト用環境変数名の変更                                                                                         

    src/backend/test/pytest/test_flower_get.py

    • 環境変数BUCKET_NAMEFLOWER_BUCKET_NAMEに変更
    +1/-1     

    💡 PR-Agent usage: Comment /help "your question" on any pull request to receive relevant information

    @mt-osho-san mt-osho-san self-assigned this Dec 29, 2024
    @github-actions github-actions bot changed the title Feature/#258 feat: S3バケット名の変更と新規バケット追加 Dec 29, 2024
    Copy link
    Contributor

    PR Reviewer Guide 🔍

    Here are some key observations to aid the review process:

    🎫 Ticket compliance analysis ✅

    258 - Fully compliant

    Fully compliant requirements:

    • S3バケットの名称をflowerImageBucketからoriginalImageBucketに変更
    • 新たにflowerBucketを追加

    Not compliant requirements:

    • なし
    ⏱️ Estimated effort to review: 3 🔵🔵🔵⚪⚪
    🔒 No security concerns identified
    🔀 No multiple PR themes
    ⚡ Recommended focus areas for review

    新規バケット追加

    flowerBucketoriginalImageBucketの追加に伴う影響範囲の確認が必要です。

    flowerBucket: flower.flowerBucket,
    userPool: auth.userPool,
    api: api.api,
    cognitoAuthorizer: api.cognitoAuthorizer,
    table: flower.table,
    generativeAiTable: flower.generativeAiTable,
    flowerSelectFunction: flower.flowerSelectFunction,
    originalImageBucket: flower.originalImageBucket,

    @mt-osho-san mt-osho-san merged commit 7d87492 into main Jan 1, 2025
    12 checks passed
    @mt-osho-san mt-osho-san deleted the feature/#258 branch January 1, 2025 02:37
    Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
    Projects
    None yet
    Development

    Successfully merging this pull request may close these issues.

    2 participants