Skip to content

Commit

Permalink
Merge pull request #61 from COW-dev/main
Browse files Browse the repository at this point in the history
  • Loading branch information
dojinyou authored Nov 20, 2023
2 parents 6e37a7e + 7f43f4a commit dace028
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 4 deletions.
3 changes: 1 addition & 2 deletions .github/workflows/deploy.yml
Original file line number Diff line number Diff line change
Expand Up @@ -38,8 +38,7 @@ jobs:
DB_URL=${{ secrets.DB_URL }}
DB_USERNAME=${{ secrets.DB_USERNAME }}
DB_PASSWORD=${{ secrets.DB_PASSWORD }}
REDIS_URL=${{ secrets.REDIS_URL }}
REDIS_PORT=${{ secrets.REDIS_PORT }}
REDIS_URL=${{ secrets.REDIS_URL }}
tags: |
eatda/api-server:latest
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -28,8 +28,9 @@ class StoreQueryService(
}

fun findById(storeId: Long): StoreDetailDto {
val entity = repository.getReferenceById(storeId)
// FIXME(async): findById가 popular store command 과정에서 실패하지 않도록 비동기 호출 처리
popularStoreCommandService.setStore(storeId)
return StoreDetailDto.from(repository.getReferenceById(storeId))
return StoreDetailDto.from(entity)
}
}
1 change: 0 additions & 1 deletion src/main/resources/application.yml
Original file line number Diff line number Diff line change
Expand Up @@ -72,7 +72,6 @@ spring:
data:
redis:
host: ${REDIS_URL}
port: ${REDIS_PORT}

jooq:
sql-dialect: postgres

0 comments on commit dace028

Please sign in to comment.