Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Comments

ํ•ด์™ธ์ฃผ์‹ ์ž”๊ณ  ์ˆ˜๋Ÿ‰ ๋ฒ„๊ทธ ์ˆ˜์ •#43

Merged
Soju06 merged 1 commit intomainfrom
fix/41-foregin-balance-stock-quantity
Aug 28, 2024
Merged

ํ•ด์™ธ์ฃผ์‹ ์ž”๊ณ  ์ˆ˜๋Ÿ‰ ๋ฒ„๊ทธ ์ˆ˜์ •#43
Soju06 merged 1 commit intomainfrom
fix/41-foregin-balance-stock-quantity

Conversation

@Soju06
Copy link
Owner

@Soju06 Soju06 commented Aug 28, 2024

๐Ÿ› ๏ธ PR Summary

๐ŸŒŸ ์š”์•ฝ

์–ด๋–ค ๊ฒƒ์ด ๋ณ€๊ฒฝ๋˜์—ˆ๋‚˜์š”? ๊ฐ„๋žตํžˆ ์„ค๋ช…ํ•ด์ฃผ์„ธ์š”.

์‹ค์ „๊ณ„์ขŒ์—์„œ ํ•ด์™ธ์ฃผ์‹์˜ ๋ณด์œ  ์ˆ˜๋Ÿ‰์ด 0์ธ ๋ฌธ์ œ๋ฅผ ํ•ด๊ฒฐํ–ˆ์Šต๋‹ˆ๋‹ค.

๐Ÿ“Š ์ฃผ์š” ๋ณ€๊ฒฝ ์‚ฌํ•ญ

์ฃผ์š” ๋ณ€๊ฒฝ ์‚ฌํ•ญ์„ ์ ์–ด์ฃผ์„ธ์š”.

  • api/account/balance.py ํŒŒ์ผ์—์„œ foreign_balance ํ•จ์ˆ˜์˜ if self.virtual:์„ ์ œ๊ฑฐํ–ˆ์Šต๋‹ˆ๋‹ค.

๐ŸŽฏ ๋ชฉ์  ๋ฐ ์˜ํ–ฅ

  • ๋ชฉ์ : ์™œ ์ด PR์ด ํ•„์š”ํ•œ๊ฐ€์š”?
    ๋ณด์œ ์ˆ˜๋Ÿ‰ ๋ฒ„๊ทธ๋ฅผ ์ˆ˜์ •ํ–ˆ์Šต๋‹ˆ๋‹ค.

  • ์˜ํ–ฅ: ์ด ๋ณ€๊ฒฝ ์‚ฌํ•ญ์ด ์–ด๋–ค ์˜ํ–ฅ์„ ๋ฏธ์น˜๋‚˜์š”?
    ์‹ค์ „ ๊ณ„์ขŒ๋„ ์ž”๊ณ  ์กฐํšŒ API๋ฅผ ํ˜ธ์ถœํ•˜๋„๋ก ๋ณ€๊ฒฝํ–ˆ์Šต๋‹ˆ๋‹ค.

@Soju06 Soju06 added the ๋ฒ„๊ทธ ๋ญ”๊ฐ€ ์ž‘๋™ํ•˜์ง€ ์•Š์Šต๋‹ˆ๋‹ค. label Aug 28, 2024
@Soju06 Soju06 self-assigned this Aug 28, 2024
@Soju06 Soju06 merged commit da36325 into main Aug 28, 2024
@Soju06 Soju06 deleted the fix/41-foregin-balance-stock-quantity branch August 28, 2024 00:17
@minari2
Copy link

minari2 commented Aug 29, 2024

๊ฐ์‚ฌํ•ฉ๋‹ˆ๋‹ค.

Soju06 added a commit that referenced this pull request Feb 20, 2026
ํ•ด์™ธ์ฃผ์‹ ์ž”๊ณ  ์ˆ˜๋Ÿ‰ ๋ฒ„๊ทธ ์ˆ˜์ •
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

๋ฒ„๊ทธ ๋ญ”๊ฐ€ ์ž‘๋™ํ•˜์ง€ ์•Š์Šต๋‹ˆ๋‹ค.

Projects

None yet

2 participants