-
Notifications
You must be signed in to change notification settings - Fork 0
Refactor: Clean up Docker, remove Elasticsearch, and fix search functionality. #219
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
Changes from all commits
4b01e3a
3aec6d5
df265bb
714c8cf
6312610
1a46c4f
0957bf9
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1 +1 @@ | ||
| db/ | ||
| docker/db/ |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -37,7 +37,7 @@ out/ | |
| .vscode/ | ||
| .idea | ||
| .env | ||
| /db | ||
| /docker/db | ||
| !/resources/db/*.sql | ||
| mongodb | ||
|
|
||
|
|
||
This file was deleted.
This file was deleted.
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,30 @@ | ||
| version: "3.8" | ||
|
|
||
| services: | ||
| db: | ||
| image: mysql:8.4 | ||
| container_name: db | ||
| platform: linux/amd64 | ||
| environment: | ||
| MYSQL_USER: artrip | ||
| MYSQL_ROOT_PASSWORD: artrip1! | ||
| MYSQL_DATABASE: artrip | ||
| MYSQL_PASSWORD: artrip1! | ||
| ports: | ||
| - "33069:3306" | ||
| volumes: | ||
| - ./db/data:/var/lib/mysql | ||
| - ./db/conf:/etc/mysql/conf.d | ||
|
|
||
| redis: | ||
| image: redis:7.2 | ||
| container_name: redis | ||
| restart: always | ||
| ports: | ||
| - "63799:6379" | ||
| volumes: | ||
| - redis-data:/data | ||
|
|
||
|
|
||
| volumes: | ||
| redis-data: |
This file was deleted.
This file was deleted.
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,16 +1,31 @@ | ||
| package org.atdev.artrip.config; | ||
|
|
||
| import lombok.RequiredArgsConstructor; | ||
| import org.atdev.artrip.global.resolver.LoginUserIdArgumentResolver; | ||
| import org.springframework.context.annotation.Configuration; | ||
| import org.springframework.web.method.support.HandlerMethodArgumentResolver; | ||
| import org.springframework.web.servlet.config.annotation.CorsRegistry; | ||
| import org.springframework.web.servlet.config.annotation.WebMvcConfigurer; | ||
|
|
||
| import java.util.List; | ||
|
|
||
| @Configuration | ||
| @RequiredArgsConstructor | ||
| public class WebConfig implements WebMvcConfigurer { | ||
|
|
||
| private final LoginUserIdArgumentResolver userIdArgumentResolver; | ||
|
|
||
| @Override | ||
| public void addCorsMappings(CorsRegistry registry) { | ||
| registry.addMapping("/**") | ||
| .allowedOrigins("http://localhost:3000", "http://localhost:8080", "https://dev.coffit.today") | ||
| .allowedMethods("OPTIONS","GET", "POST", "PUT", "PATCH", "DELETE") | ||
| .allowCredentials(true); | ||
| } | ||
|
|
||
| @Override | ||
| public void addArgumentResolvers(List<HandlerMethodArgumentResolver> resolvers) { | ||
| resolvers.add(userIdArgumentResolver); | ||
| } | ||
|
|
||
| } |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -17,7 +17,6 @@ | |
| @RestController | ||
| @RequiredArgsConstructor | ||
| @RequestMapping("/admin/exhibits") | ||
| @Slf4j | ||
| @Tag(name = "Admin - Exhibit", description = "관리자 전시 관리 API") | ||
| public class AdminExhibitController { | ||
|
|
||
|
|
@@ -37,7 +36,6 @@ public class AdminExhibitController { | |
| ) | ||
| @GetMapping | ||
| public CommonResponse<PagingResponseDTO<AdminExhibitListResponse>> getExhibitList(Criteria cri) { | ||
| log.info("Admin getting exhibit list: {}", cri); | ||
|
|
||
| PagingResponseDTO<AdminExhibitListResponse> result = adminExhibitService.getExhibitList(cri); | ||
|
|
||
|
|
@@ -47,7 +45,6 @@ public CommonResponse<PagingResponseDTO<AdminExhibitListResponse>> getExhibitLis | |
| @Operation(summary = "전시 상세 조회", description = "특정 전시의 상세 정보를 조회합니다.") | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 문서화 어노테이션 분리 필요
Collaborator
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 우경님이 분리된 내용 공유 해줘서 그 내용기반으로 수정하도록 하겠습니다. |
||
| @GetMapping("/{exhibitId}") | ||
| public CommonResponse<AdminExhibitResponse> getExhibit(@PathVariable Long exhibitId) { | ||
| log.info("Admin getting exhibit : {}", exhibitId); | ||
|
|
||
| AdminExhibitResponse result = adminExhibitService.getExhibit(exhibitId); | ||
|
|
||
|
|
@@ -57,7 +54,6 @@ public CommonResponse<AdminExhibitResponse> getExhibit(@PathVariable Long exhibi | |
| @Operation(summary = "전시 등록", description = "새로운 전시를 등록합니다.") | ||
| @PostMapping | ||
| public CommonResponse<Long> createExhibit(@RequestBody CreateExhibitRequest request) { | ||
| log.info("Admin creating exhibit: title = {}", request.getTitle()); | ||
|
|
||
| Long exhibitId = adminExhibitService.createExhibit(request); | ||
|
|
||
|
|
@@ -67,7 +63,6 @@ public CommonResponse<Long> createExhibit(@RequestBody CreateExhibitRequest requ | |
| @Operation(summary = "전시 수정", description = "특정 전시를 수정합니다.") | ||
| @PutMapping("/{exhibitId}") | ||
| public CommonResponse<Long> updateExhibit(@PathVariable Long exhibitId, @RequestBody UpdateExhibitRequest request){ | ||
| log.info("Admin updating exhibit: {}", request.getTitle()); | ||
|
|
||
| Long updatedId = adminExhibitService.updateExhibit(exhibitId, request); | ||
|
|
||
|
|
@@ -77,7 +72,6 @@ public CommonResponse<Long> updateExhibit(@PathVariable Long exhibitId, @Request | |
| @Operation(summary = "전시 삭제", description = "특정 전시를 삭제합니다.") | ||
| @DeleteMapping("/{exhibitId}") | ||
| public CommonResponse<Void> deleteExhibit(@PathVariable Long exhibitId) { | ||
| log.info("Admin deleting exhibit: {}", exhibitId); | ||
|
|
||
| adminExhibitService.deleteExhibit(exhibitId); | ||
| return CommonResponse.onSuccess(null); | ||
|
|
||
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.
응답 스펙 관련해서 정해진 내용이 있다면 그걸로 변경해주세요
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.
넵