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

상품 조회 오류 수정, 푸시 알림 shop 정보 추가, URL 검증 api 분리 #246

Merged
merged 4 commits into from
Feb 5, 2024

Conversation

Muungi
Copy link
Collaborator

@Muungi Muungi commented Feb 5, 2024

진행 내용

-[x] 상품 조회 오류 수정
-[x] 푸시 알림 shop 정보 추가
-[x] URL 검증 api 분리

  • cron 과정에서 변수 명이 불일치 하여 오류가 발생하는 이슈를 수정하였습니다.
  • 푸시 알림에서도 11번가와 SmartStore를 구분하기 위해 shop 정보를 추가하였습니다.
  • URL 검증 과정에서 SmartStore를 지원하지 않는 어플에서 SmartStore 상품을 등록하는 경우, 11번가와 상품코드가 겹치면 잘 못 등록되는 오류가 있었습니다.
  • 해당 문제로 인해 검증 URL도 버전을 분리하였습니다.

@Muungi Muungi added feature 새로운 기능 bug 버그 수정 backend 백엔드 관련 labels Feb 5, 2024
@Muungi Muungi requested a review from sickbirdd February 5, 2024 13:57
@Muungi Muungi self-assigned this Feb 5, 2024
Copy link
Collaborator

@sickbirdd sickbirdd left a comment

Choose a reason for hiding this comment

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

미처 확인하지 못했던 자잘한 버그들을 문기님께서 잘 캐치하셔서 수정해주신 것 같아요~
정말 고생하셨습니다!

backend/src/cron/cron.service.ts Show resolved Hide resolved
backend/src/cron/cron.service.ts Show resolved Hide resolved
backend/src/cron/cron.service.ts Outdated Show resolved Hide resolved
@sickbirdd sickbirdd self-requested a review February 5, 2024 14:23
@sickbirdd sickbirdd merged commit c01bdb9 into dev/be Feb 5, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend 백엔드 관련 bug 버그 수정 feature 새로운 기능
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants