Skip to content

!HOTFIX(client): 리마인드 카드 시간 로직 변경#178

Merged
constantly-dev merged 5 commits intodevelopfrom
fix/#175/remind-time-logic-change
Oct 2, 2025
Merged

!HOTFIX(client): 리마인드 카드 시간 로직 변경#178
constantly-dev merged 5 commits intodevelopfrom
fix/#175/remind-time-logic-change

Conversation

@constantly-dev
Copy link
Member

@constantly-dev constantly-dev commented Oct 2, 2025

📌 Related Issues

관련된 Issue를 태그해주세요. (e.g. - close #25)

📄 Tasks

  • 리마인드 카드 시간 로직 변경

⭐ PR Point (To Reviewer)

기획과 개발팀 싱크가 안맞아서 로직 변경했습니다.

AS-IS

카드 남은 시간 = 리마인드 설정 시간 - 현재 시간

TO-BE

카드 남은 시간 = 리마인드 시간으로부터 남은 24시간 표시
크게 많이 바뀐 것은 없습니다! 약간씩만 바꾸면 가능해서 HOTFIX로 수정했습니다.

📷 Screenshot

image

Summary by CodeRabbit

  • 버그 수정
    • 리마인드 글 노출 기간을 “리마인드 시간부터 24시간 동안”으로 정확히 적용하여 조기/지연 노출을 방지했습니다.
    • 리마인드 카드의 남은 시간 계산을 새 24시간 기준에 맞춰 정확히 표시합니다.
  • Chores
    • 토큰 갱신 실패 시 저장된 이메일을 더 이상 삭제하지 않아, 웹과 확장 프로그램에서 온보딩으로 이동하더라도 이메일이 유지되어 재로그인이 편리해졌습니다. 토큰만 초기화됩니다.

@vercel
Copy link

vercel bot commented Oct 2, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
pinback-client-client Ready Ready Preview Comment Oct 2, 2025 5:03pm
pinback-client-landing Ready Ready Preview Comment Oct 2, 2025 5:03pm

@github-actions github-actions bot added the fix 버그 수정하라 러브버그 label Oct 2, 2025
@coderabbitai
Copy link

coderabbitai bot commented Oct 2, 2025

Warning

Rate limit exceeded

@constantly-dev has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 18 minutes and 48 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

📥 Commits

Reviewing files that changed from the base of the PR and between 862accd and 3f96a0d.

📒 Files selected for processing (2)
  • apps/client/src/shared/components/cardEditModal/CardEditModal.tsx (1 hunks)
  • apps/extension/src/pages/MainPop.tsx (1 hunks)

Walkthrough

리마인드 표시 기간을 “remindAt부터 24시간”으로 재정의하고, 카드의 남은 시간 계산을 이에 맞춰 변경했습니다. 토큰 재발급 실패 시 localStorage에서 email을 삭제하던 로직을 클라이언트/익스텐션 axios 인스턴스에서 제거했습니다. 공개 API 시그니처 변화는 없습니다.

Changes

Cohort / File(s) Summary
Remind 시간 윈도우 재정의
apps/client/src/pages/remind/Remind.tsx
표시 조건을 “now ∈ [remindAt, remindAt+24h)”로 변경, expirationTime 도입, 기존 displayTimeLimit 제거 및 관련 주석 삭제
RemindCard 남은시간 계산 맞춤
packages/design-system/src/components/card/RemindCard.tsx
endTime을 remindTime+24h 기준으로 재계산하여 카운트다운/만료 판단 로직 조정
토큰 재발급 실패 시 email 보존 (Client)
apps/client/src/shared/apis/setting/axiosInstance.ts
요청/응답 인터셉터의 재발급 실패 경로에서 localStorage.removeItem('email') 제거, token만 삭제 및 온보딩 리다이렉트 유지
토큰 재발급 실패 시 email 보존 (Extension)
apps/extension/src/apis/axiosInstance.ts
재발급 실패 경로에서 localStorage.removeItem('email') 제거, 나머지 흐름 동일

Sequence Diagram(s)

sequenceDiagram
  autonumber
  actor User
  participant RemindPage as Remind Page
  participant Store as Data Store

  User->>RemindPage: Open Remind
  RemindPage->>Store: Fetch remind items
  Store-->>RemindPage: Items(remindAt)
  note over RemindPage: Now ∈ [remindAt, remindAt+24h)
  RemindPage-->>User: Render items filtered by new 24h window
Loading
sequenceDiagram
  autonumber
  participant App as App (Client/Extension)
  participant Axios as AxiosInstance
  participant API as Backend
  participant LS as localStorage

  App->>Axios: Request
  Axios->>API: Send with token
  API-->>Axios: 401 Unauthorized
  Axios->>API: Refresh token
  alt refresh fails
    Axios->>LS: removeItem('token')
    note right of Axios: email 보존 (remove 중단)
    Axios-->>App: Redirect to onboarding + error
  else refresh succeeds
    Axios->>API: Retry with new token
    API-->>Axios: 200 OK
    Axios-->>App: Response
  end
Loading

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Suggested reviewers

  • jllee000
  • jjangminii

Poem

토끼는 시계를 톡, 하루의 창을 딱!
remindAt에서 시작해 24시간 깔끔딱! ⏰
카드엔 남은 시간이 살포시 흘러가고,
토큰 풀썩 쓰러져도 이메일은 안녕하고.
깡총—버그는 저리 가, 오늘도 핀백 굿잡! 🥕

Pre-merge checks and finishing touches

❌ Failed checks (2 warnings)
Check name Status Explanation Resolution
Linked Issues Check ⚠️ Warning PR에서 리마인드 카드 시간 로직 변경(issue #175)은 올바르게 구현되었으나 linked issues에 포함된 progress bar 구현(issue #25)에 대한 어떠한 변경 사항이나 코드가 전혀 반영되지 않아 관련 요구사항이 충족되지 않고 있습니다. Progress 컴포넌트 구현(issue #25) 요구사항을 제거하거나 해당 기능을 별도의 PR로 분리하여 처리해야 합니다.
Out of Scope Changes Check ⚠️ Warning axiosInstance 파일에서 localStorage의 이메일 제거 로직을 삭제한 변경은 리마인드 카드 시간 로직 변경(issue #175)과 연관성이 없으며 scope를 벗어난 수정사항으로 보입니다. 해당 axiosInstance 관련 변경을 별도의 PR로 분리하거나 원복하여 scope를 명확히 분리해야 합니다.
✅ Passed checks (3 passed)
Check name Status Explanation
Title Check ✅ Passed PR 제목이 리마인드 카드의 시간 로직 변경이라는 핵심 내용을 명확하고 간결하게 요약하고 있으며 불필요한 세부나 모호한 표현 없이 주된 변경사항을 잘 반영하고 있습니다.
Description Check ✅ Passed PR 설명이 레포지토리 템플릿의 Related Issues, Tasks, PR Point, Screenshot 섹션을 모두 포함하여 요구된 형식을 충실히 따르고 있으며 필요한 정보가 명확하게 기술되어 있습니다.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@constantly-dev constantly-dev requested review from jjangminii and removed request for jjangminii October 2, 2025 16:54
@github-actions
Copy link

github-actions bot commented Oct 2, 2025

✅ Storybook chromatic 배포 확인:
🐿️ storybook

@constantly-dev constantly-dev added fix 버그 수정하라 러브버그 and removed fix 버그 수정하라 러브버그 labels Oct 2, 2025
Copy link
Collaborator

@jjangminii jjangminii left a comment

Choose a reason for hiding this comment

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

고생하셨습니다~~ 확실히 의도된대로 동작중이었지만 기획팀과 소통 오류가 있었네요..

Copy link
Collaborator

@jllee000 jllee000 left a comment

Choose a reason for hiding this comment

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

당신은 QA의 신??

@constantly-dev constantly-dev merged commit 9cafc14 into develop Oct 2, 2025
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fix 버그 수정하라 러브버그

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[!HOTFIX] 리마인드 카드 시간 로직 변경

3 participants