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

[fix] appid type hotfix #118

Merged
merged 1 commit into from
Feb 29, 2024

Conversation

mediwind
Copy link
Contributor

@mediwind mediwind commented Feb 29, 2024

작업 분류

  • 신규 기능
  • 기능 수정
  • 버그 수정
  • 프로젝트 구조 변경

작업 상세 내용

appid → int(appid)

생각해볼 문제

resolve #{issue number}

@srlee056
Copy link
Contributor

appid를 int로 바꿔야 하는 이유가 있나요?

@mediwind mediwind merged commit db86cf7 into develop Feb 29, 2024
3 checks passed
@mediwind
Copy link
Contributor Author

헉 안주무셨나요

캡처

price 정보가 제대로 오지 않는 경우 "steam_appid" 키의 값이 문자열로 도착해서
혹시나 해서 price 정보가 정상적으로 올 때 처럼 값이 int로 도착하도록 했습니다.

혹시나 해서요

@srlee056
Copy link
Contributor

아 이미 merge하셨긴 한데
steam_appid 그러면 raw_data 형태에서는 int/string 형태가 섞여있는거네요? 그래서 어떤건 int로 어떤건 string으로 인식됐던거구나
혹시 모르니까 script에서 전부 다 string으로 cast하도록 수정은 해두겠습니다

@mediwind mediwind deleted the develop_for_adding_region_lock_defense_code branch February 29, 2024 14:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants