Skip to content

2024년 12월 비정기배포 가이드 (클라)#350

Open
phillip5094 wants to merge 4 commits into
alpha-client-20241210-basefrom
alpha-client-20241210-pr
Open

2024년 12월 비정기배포 가이드 (클라)#350
phillip5094 wants to merge 4 commits into
alpha-client-20241210-basefrom
alpha-client-20241210-pr

Conversation

@phillip5094
Copy link
Copy Markdown
Contributor

No description provided.

Comment thread ko/release-notes-ios.md
[SDK Download](https://static.toastoven.net/toastcloud/sdk_download/gamebase/v2.68.1/GamebaseSDK-iOS.zip)

#### 버그 수정
* Swift 파일에서 Gamebase SDK를 import할 때 발생하던 오류를 수정하였습니다.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
* Swift 파일에서 Gamebase SDK를 import할발생하던 오류를 수정하였습니다.
* Swift 파일에서 Gamebase SDK를 내보낼발생하는 오류를 수정하였습니다.

의미가 바뀌거나 어색해지지 않을 경우 가급적 국어 표현을 쓰고자 하여 수정 의견을 드립니다.

Copy link
Copy Markdown
Contributor Author

@phillip5094 phillip5094 Dec 10, 2024

Choose a reason for hiding this comment

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

@tw-sunyoung
범용적으로 import 라는 단어를 많이 사용하고 국어 표현을 쓰고자 한다면 '추가' 또는 '사용' 한다의 의미로 봐야할 것 같습니다.

import는 유지하고 아래처럼 수정하는 어떨까요?

Suggested change
* Swift 파일에서 Gamebase SDK를 import할 때 발생하던 오류를 수정하였습니다.
* Swift 파일에서 Gamebase SDK를 import할 때 발생하는 오류를 수정하였습니다.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@phillip5094 전임님, 제가 export로 순간 착각하여 잘못 의견을 드렸네요. 죄송합니다.
한글로 적용하실 경우 '가져올 때' 정도로 의견 드리며, 말씀하신 바와 같이 자주 사용하는 표현이므로 여기서는 import로 유지해 주셔도 무방할 것 같습니다. 감사합니다!

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@tw-sunyoung
넵 검토 감사합니다. 'import' 용어는 유지하고 그외 검수 의견은 반영하도록 하겠습니다.
감사합니다.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants