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] 기획 변경에 따른 익스텐션 변경 #919

Merged
merged 17 commits into from
Jan 7, 2025

Conversation

dmdgpdi
Copy link
Collaborator

@dmdgpdi dmdgpdi commented Jan 6, 2025

  • Close #ISSUE_NUMBER

What is this PR? 🔍

먼저 기획이 변경되었습니다. 바뀐 기획은 다음과 같습니다.

  • 모든 북마크는 미분류 폴더로 바로 이동합니다. (중복 생성이 가능합니다.)
  • 동기적으로 ogData를 불러왔던 부분이 변경됩니다. ogImage는 백엔드에서 비동기적으로 가져오고 익스텐션의 UI는 파비콘을 이용합니다. (og image가 없더라도 비동기적으로 백엔드에서 파비콘을 가져옵니다.)
스크린샷 2025-01-06 오후 2 19 19
  • 페이지마다 url마다 방문 여부에 따라 아이콘이 변경됩니다.
스크린샷 2025-01-06 오후 2 20 24 스크린샷 2025-01-06 오후 2 20 17

Changes 📝

  • API 호출을 Promise.all을 이용해 한번에 호출합니다.
  • 기획 변경으로 더이상 사용하지 않는 코드를 삭제합니다.

Precaution

  • 중복과 비동기 처리에 대해서 어떤 버그가 생성될 지 모르니 개발 서버에서 사용한 후, 문제를 발견하지 않으면 프로덕션으로 배포합니다.

@dmdgpdi dmdgpdi added feat 기능 frontend 프론트 관련 D-3 PR 등록일 기준 3일 이내 확인 필요 labels Jan 6, 2025
@dmdgpdi dmdgpdi self-assigned this Jan 6, 2025
@dmdgpdi dmdgpdi merged commit 0739edb into fe-develop Jan 7, 2025
1 check passed
@dmdgpdi dmdgpdi deleted the front/feat/#902 branch January 9, 2025 03:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
D-3 PR 등록일 기준 3일 이내 확인 필요 feat 기능 frontend 프론트 관련
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant