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.

Sign up for GitHub

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

Jump to bottom

Hotfix#/캐시로 인해 페이지가 나오지 않던 문제 수정, 배치업데이트 버그 수정 #173

Merged
merged 3 commits into from
Aug 30, 2024
Merged

Hotfix#/캐시로 인해 페이지가 나오지 않던 문제 수정, 배치업데이트 버그 수정 #173

merged 3 commits into from
Aug 30, 2024

Conversation

bellringstar
Copy link
Collaborator

📄 작업 설명

  • 새로 생성해도 기존 캐시 정보때문에 페이지에서 나오지 않던 문제를 수정했습니다.
  • 티켓 재고 벌크 업데이트 시 created_at, updated_at이 없어 실패하던 문제를 수정했습니다.

🚨 관련 이슈

closes #[이슈번호]

🌈 작업 상황

📌 기타

@bellringstar bellringstar added the 버그 Something isn't working label Aug 30, 2024
@bellringstar bellringstar requested a review from a team August 30, 2024 00:18
@bellringstar bellringstar self-assigned this Aug 30, 2024
@netlify Netlify
Copy link

netlify bot commented Aug 30, 2024 •
edited
Loading

✅ Deploy Preview for twodari ready!

Name Link
🔨 Latest commit 8c1df7c
🔍 Latest deploy log https://app.netlify.com/sites/twodari/deploys/66d10fba2a74a70008b3e580
😎 Deploy Preview https://deploy-preview-173--twodari.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@github-actions GitHub Actions
Copy link

Risk Level 2 - /home/runner/work/Team2-Dari/Team2-Dari/backend/api-server/src/main/java/com/wootecam/festivals/domain/ticket/service/TicketService.java

  1. The use of @CacheEvict is appropriate for clearing the cache when a new ticket is created. Ensure that the cache name 'ticketList' is consistent with other parts of the application. 2. Consider adding logging for the ticket creation process to help with debugging in case of failures.

Risk Level 2 - /home/runner/work/Team2-Dari/Team2-Dari/backend/api-server/src/main/java/com/wootecam/festivals/domain/ticket/repository/TicketStockJdbcRepository.java

  1. The addition of the Timestamp for created_at and updated_at is good for tracking. Ensure that the database schema matches this expectation. 2. Consider using a more precise time source, such as Instant.now(), for better accuracy.

Risk Level 2 - /home/runner/work/Team2-Dari/Team2-Dari/frontend/src/pages/admin/TicketManagement.js

  1. The error handling in handleCreateTicket is improved by logging the specific error message. Ensure that the error response structure is consistent across the API. 2. Consider using a more user-friendly error message for the UI instead of directly displaying the API error.

🛠️🔍📈

Powered by Code Review GPT

@bellringstar bellringstar merged commit 1916c9e into develop Aug 30, 2024
7 checks passed
@bellringstar bellringstar deleted the hotfix branch August 30, 2024 00:18
@github-actions GitHub Actions
Copy link

Code Coverage Report

Overall Project 89.53% 🍏
Files changed 100% 🍏

Module Coverage
api-server 88.78% 🍏
Files
Module File Coverage
api-server FestivalService.java 100% 🍏
TicketStockJdbcRepository.java 100% 🍏
TicketService.java 100% 🍏

bellringstar added a commit that referenced this pull request Aug 31, 2024
@bellringstar @minnim1010
docs#171/README.md 작성 (#172)
04fc039 
* docs: README.md 초안 작성

* hotfix: 레디스 설정 중복으로 로딩 실패하는 버그 해결

* Hotfix#/캐시로 인해 페이지가 나오지 않던 문제 수정, 배치업데이트 버그 수정 (#173)

* fix: 티켓, 축제 생성 시 캐시 무효화

* fix: 티켓 재고 생성시 created_at, updated_at 추가

* fix: 에러메시지 구체화

---------

Co-authored-by: bellringstar <[email protected]>

* hotfix: 티켓 생성 후 구매 요청 시 티켓 정보 없음 문제 해결

- 티켓 생성 시 레디스에 티켓 정보 넣는 로직 추가

* fix: 로그를 계속 읽어 큐가 가득차는 버그 수

* hotfix: 티켓 생성 후 구매 요청 시 티켓 정보 없음 문제 해결

- 10분 내 판매되는 티켓 생성 시 레디스에 티켓 정보 넣는 로직 추가

* fix: 티켓 디테일 api 수정

* fix: 티켓 참여 여부 화면 수정

* fix: qr 체크인 이후 구매자 페이지로 이동하는 기능 제거

* fix: 페이지네이션 버그 수

* fix: 페이지네이션 버그 수정

* fix: 페이지네이션 조건 수

* fix: 체크인 여부 페이지 수정

* fix: festival 엔티티 시간 저장 방식 변

* fix: 재고 부족 페이지 추가

* hotfix: 페이지네이션 쿼리 수

* hotfix: 페스티벌 저장 방식 변경

- 시작 시간을 초단위로 저

* hotfix: 대기순번 로컬스토리지에 저장하도록 수정

* docs: README.md 초안 작성

* docs: README 작성

---------

Co-authored-by: bellringstar <[email protected]>
Co-authored-by: minnim1010 <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@minnim1010 minnim1010 minnim1010 approved these changes

@kkyu0718 kkyu0718 kkyu0718 approved these changes

Assignees

@bellringstar bellringstar

Labels
버그 Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@bellringstar @minnim1010 @kkyu0718