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

Feat/#158 get poke to me single #166

Merged
merged 21 commits into from
Dec 10, 2023

Conversation

yummygyudon
Copy link
Member

๐Ÿ“ PR Summary

๋‚˜๋ฅผ ์ฐŒ๋ฅธ ์นœ๊ตฌ ์กฐํšŒ API ๊ตฌํ˜„

๐ŸŒต Working Branch

feat/#158-get-poke-to-me-single

์žฌ์‚ฌ์šฉํ•  ์ˆ˜ ์žˆ๋Š” ๋ถ€๋ถ„์ด ๋งŽ์•„
๋ธŒ๋žœ์น˜ ์ด๋ฆ„์€ single(๋‹จ์ผ) ์กฐํšŒ๋ฅผ ์œ„ํ•ด ์ƒ์„ฑํ–ˆ์—ˆ์ง€๋งŒ
๋ฆฌ์ŠคํŠธ ์กฐํšŒ๊นŒ์ง€ ๊ตฌํ˜„ํ–ˆ์Šต๋‹ˆ๋‹ค!

๐ŸŒด Works

  • Playground Internal API ์ˆ˜์ • ๋ฐ˜์˜
  • [ ]

๐ŸŒฑ Related Issue

#158
#159

@yummygyudon yummygyudon requested a review from gunom December 9, 2023 16:55
@yummygyudon yummygyudon self-assigned this Dec 9, 2023
@yummygyudon yummygyudon changed the base branch from dev to sprint2/poke December 9, 2023 16:55
- ์›์ธ : `java.lang.IllegalStateException: For queries with named parameters you need to use provide names for method parameters. Use @param for query method parameters, or when on Java 8+ use the javac flag -parameters.` ์—๋Ÿฌ ๋ฐœ์ƒ
- ํ™•์ธ ํ•„์š” ์‚ฌํ•ญ : `getRecommendFriendsOfUsersFriend` ๋ฉ”์„œ๋“œ ๋‚ด๋ถ€ friendIds ์— ๋Œ€ํ•œ Mapping์—์„œ null๊ฐ’์ด ๋“ค์–ด๊ฐ€๊ฒŒ๋˜๋Š” ๊ฒฝ์šฐ๊ฐ€ ๋ฐœ์ƒ
  - ์ด๋ ‡๊ฒŒ ๋  ๊ฒฝ์šฐ, Playground API ํ†ต์‹ ํ•  ๋•Œ `memberIds=` ํŒŒ๋ผ๋ฏธํ„ฐ์— ""์ด ๋“ค์–ด๊ฐ€๋Š” ๊ฒฝ์šฐ๊ฐ€ ๋ฐœ์ƒํ•  ์ˆ˜๋„ ์žˆ์Œ.
  - ํ•˜์ง€๋งŒ ํ•ด๋‹น ๋ฉ”์„œ๋“œ ํ˜ธ์ถœ์‹œ์ ์ด `memberIds`์— ๊ฐ’์ด ๋ฌด์ €๊ฑด ๋“ค์–ด๊ฐ€๋Š” ๊ฒฝ์šฐ๋ผ๋ฉด ๋ฌธ์ œ์—†์„ ๋“ฏ
Copy link
Contributor

@gunom gunom left a comment

Choose a reason for hiding this comment

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

๋ฌธ์ œ ๋  ๊ฒƒ ์—†์„ ๊ฒƒ ๊ฐ™์•„์š”! ๊ฐ์‚ฌํ•ฉ๋‹ˆ๋‹ค

@yummygyudon yummygyudon merged commit a99c6ad into sprint2/poke Dec 10, 2023
@ozzing ozzing deleted the feat/#158-get-poke-to-me-single branch May 23, 2024 16:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

feat : ๋‚˜๋ฅผ ์ฐŒ๋ฅธ ์‚ฌ๋žŒ ์กฐํšŒ (๋‹จ์ผ)
2 participants