Skip to content

๐Ÿ”ฅ Remove localstorage regId after life cycle - #189

Merged
young-52 merged 13 commits into
devfrom
feat/regId-remove
Jun 26, 2026
Merged

young-52 merged 13 commits into
devfrom
feat/regId-remove

Conversation

@jun-0411

Copy link
Copy Markdown
Collaborator

๐Ÿ“ ์ž‘์—… ๋‚ด์šฉ

๋น„๋กœ๊ทธ์ธ ์œ ์ €์˜ ์ผ์ • ์ฐธ์—ฌ ์‹ ์ฒญ์œผ๋กœ ์ธํ•ด ์ƒ๊ธด regId๊ฐ€ ์‚ฌ๋ผ์ง€์ง€ ์•Š๊ณ  ์˜ํ–ฅ์„ ์ฃผ๋Š” ๋ฌธ์ œ๋ฅผ ํ•ด๊ฒฐํ•˜๊ธฐ ์œ„ํ•ด regId์˜ ์ƒ๋ช…์ฃผ๊ธฐ๊ฐ€ ๋๋‚˜๋ฉด ์‚ญ์ œํ•˜๋„๋ก ์ˆ˜์ •ํ–ˆ์Šต๋‹ˆ๋‹ค.

์ž์„ธํ•œ regId ์‚ญ์ œ ๊ธฐ์ค€์€ ๋‹ค์Œ๊ณผ ๊ฐ™์Šต๋‹ˆ๋‹ค.

  • ๋กœ๊ทธ์ธ / ๋กœ๊ทธ์•„์›ƒ์‹œ ๋ชจ๋“  regId๋ฅผ ์ œ๊ฑฐํ•ฉ๋‹ˆ๋‹ค. ์ถ”๊ฐ€๋กœ ๋กœ๊ทธ์ธ ์ƒํƒœ์—์„œ๋Š” ๋กœ์ปฌ์Šคํ† ๋ฆฌ์ง€์™€ Url์˜ regId๋ฅผ ๋ฌด์‹œํ•ฉ๋‹ˆ๋‹ค.
  • ๋น„๋กœ๊ทธ์ธ ์œ ์ €๊ฐ€ ์‹ ์ฒญ์„ ์ทจ์†Œํ•˜๋ฉด ํ•ด๋‹น ์ด๋ฒคํŠธ์˜ regId๋ฅผ ์ œ๊ฑฐํ•˜๊ณ , Url์˜ ?regId=๋„ ์ œ๊ฑฐํ•ฉ๋‹ˆ๋‹ค.
  • getRegistrationDetail(regId)๊ฐ€ 401, 403, 404๋กœ ์‹คํŒจํ•˜๋ฉด ์˜ค๋ž˜๋œ ๋กœ์ปฌ ์ƒํƒœ๋ฅผ ๋ง‰๊ธฐ ์œ„ํ•ด ๋กœ์ปฌ์Šคํ† ๋ฆฌ์ง€์™€ Url์˜ regId๋ฅผ ์ œ๊ฑฐํ•ฉ๋‹ˆ๋‹ค.
  • ํ•œ ์ด๋ฒคํŠธ์—์„œ 2๊ฐœ ์ด์ƒ์˜ ๋น„๋กœ๊ทธ์ธ ์‹ ์ฒญ์ด ๋ฐœ์ƒํ•œ ๊ฒฝ์šฐ, ๊ธฐ์กด regId๋ฅผ ์ œ๊ฑฐํ•˜๊ณ  ๊ฐ€์žฅ ์ตœ์‹ ์˜ regId๋งŒ ์œ ์ง€ํ•ฉ๋‹ˆ๋‹ค.
  • ์ฃผ์ตœ์ž์˜ ๊ฐ•์ œ์ทจ์†Œ๋กœ ์ธํ•ด BANNED๊ฐ€ ๋˜๋Š” ๊ฒฝ์šฐ์—๋Š” ๋‹ค์‹œ ์‹ ์ฒญ ๋ถˆ๊ฐ€๋Šฅํ•˜๋‹ค๋Š” ํŽ˜์ด์ง€๋ฅผ ํ‘œ์‹œํ•˜๊ธฐ ์œ„ํ•ด regId๋ฅผ ์‚ญ์ œํ•˜์ง€ ์•Š๊ณ  ์œ ์ง€ํ•ฉ๋‹ˆ๋‹ค.

๐Ÿ“ธ ์Šคํฌ๋ฆฐ์ƒท (์„ ํƒ)

๐Ÿš€ ๋ฆฌ๋ทฐ ์š”๊ตฌ์‚ฌํ•ญ (์„ ํƒ)

  • dev๊ฐ€ ์•„๋‹Œ โœจ Syncronize event registration state without refreshย #187 ์˜ ๋ธŒ๋žœ์น˜ ์œ„์—์„œ ์ž‘์—…ํ•˜์—ฌ ์ด pr ๋จธ์ง€ ์ด์ „ 187์„ ์šฐ์„ ์ ์œผ๋กœ ๋จธ์ง€ํ•ด์•ผ ํ•ฉ๋‹ˆ๋‹ค.
  • ๐Ÿ’„ Fix double header in mobile uiย #171 pr์ด ๋จธ์ง€๋˜๋ฉด 187๊ณผ ์ด pr ๋จธ์ง€ ์ปจํ”Œ๋ฆญํŠธ ํ•ด๊ฒฐํ•˜๊ณ  ๋‹ค์‹œ ๋ฆฌ๋ทฐ์š”์ฒญ ๋“œ๋ฆฌ๊ฒ ์Šต๋‹ˆ๋‹ค!

This comment was marked as off-topic.

@jun-0411

Copy link
Copy Markdown
Collaborator Author

์ œ๊ฐ€ ์‚ฌ์šฉํ•˜๊ณ  ์žˆ๋Š” Agents.md ํŒŒ์ผ ์˜ฌ๋ ธ์Šต๋‹ˆ๋‹ค!

์ด์ƒํ•œ ๋ถ€๋ถ„ ์žˆ์œผ๋ฉด ํŽธํ•˜๊ฒŒ ์ˆ˜์ •ํ•˜๋ฉด์„œ ์‚ฌ์šฉํ•ด์ฃผ์„ธ์š” ใ…Žใ…Ž

@young-52 young-52 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM!

@young-52
young-52 merged commit dd463ac into dev Jun 26, 2026
1 check passed
@young-52
young-52 deleted the feat/regId-remove branch June 26, 2026 07:03
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.

๐Ÿ”ฅ Remove localstorage regId after life cycle

3 participants