-
Notifications
You must be signed in to change notification settings - Fork 1
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
장소 신고하기 API 구현 #370
장소 신고하기 API 구현 #370
Conversation
@@ -598,6 +598,27 @@ paths: | |||
required: | |||
- placeAccessibilityComment | |||
|
|||
/reportAccessibility: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
기획에 대해 잘 모르는데, reportPlaceAccessibility랑 reportBuildingAccessibility가 구분되지 않아도 괜찮나요?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
일단 지금 앱 구현에서는 따로 구분되어 있는거 같지 않아서 Place 로 합쳐놨는데 한번 여쭤볼게요!
package club.staircrusher.accessibility.application.port.out | ||
|
||
// TODO: push 기능 생기면 그거랑 합치기 | ||
interface NotificationService { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
notification이 꽤나 여러가지 의미를 가지는 것 같아서, 걍 SlackService가 더 나을지도? ㅋㅋ
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
properties: SlackNotificationProperties | ||
) : NotificationService { | ||
private val logger = KotlinLogging.logger {} | ||
private val slackService = createExternalApiService<SlackService>( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
오 감사합니다 ㅋㅋ
Checklist