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

aws 관련 configuration import 수정 및 패키지 이동 #753

Merged
merged 2 commits into from
Nov 22, 2023

Conversation

kwonyj1022
Copy link
Collaborator

📄 작업 내용 요약

aws 관련 configuration import 수정 및 패키지 이동

AwsConfiguration 에서 import를 이상한걸 import 했더라고요? 근데 테스트 환경에서는 AwsConfiguration을 사용 안 하게 해서 테스트로 못잡았던 것 같습니다. ㅠ

그래서 서버 들어가서 vim으로 변경하고 확인하던 와중에 보니까 패키지도 auction 패키지에 있더라구요..

🙋🏻 리뷰 시 주의 깊게 확인해야 하는 코드

📎 Issue 번호

@kwonyj1022 kwonyj1022 added bug 오류 발생 시 backend 백엔드와 관련된 이슈나 PR에 사용 labels Nov 17, 2023
@kwonyj1022 kwonyj1022 added this to the 레벨 5 milestone Nov 17, 2023
@kwonyj1022 kwonyj1022 self-assigned this Nov 17, 2023
Copy link
Collaborator

@apptie apptie left a comment

Choose a reason for hiding this comment

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

??? 이런 버그가 있을줄이야?

approve 하겠습니다

Copy link
Collaborator

@swonny swonny left a comment

Choose a reason for hiding this comment

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

저라면 진짜 전혀 못 찾았을 것 같은 버그.. 역시 엔초!👍🏻

Copy link
Member

@JJ503 JJ503 left a comment

Choose a reason for hiding this comment

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

import 문제가 있었을 줄이야..!
문제 원인 찾느라 고생 많으셨어요 엔초!

@kwonyj1022 kwonyj1022 merged commit f913455 into develop-be Nov 22, 2023
1 check passed
@kwonyj1022 kwonyj1022 deleted the fix/752 branch November 22, 2023 01:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend 백엔드와 관련된 이슈나 PR에 사용 bug 오류 발생 시
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

4 participants