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.

Sign up for GitHub

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

Jump to bottom

[BUG] 배너를 닫고 다른 페이지로 갔다가 홈으로 들어와있을 때 계속 열려있어요 #707

Closed
inyeong-kang opened this issue Oct 4, 2023 · 1 comment · Fixed by #787
Closed

[BUG] 배너를 닫고 다른 페이지로 갔다가 홈으로 들어와있을 때 계속 열려있어요 #707

inyeong-kang opened this issue Oct 4, 2023 · 1 comment · Fixed by #787
Assignees
@Gilpop8663
Labels
FE 🐛 버그 🚨 우선 순위 높음
Milestone
6차 데모데이

Comments

@inyeong-kang
Copy link
Member

버그 사항

배너를 닫고 다른 페이지로 갔다가 홈으로 들어와 있을 때 계속 열려있어 불편합니다.

버그 simulation

  1. 배너를 취소버튼을 눌러 닫는다.
  2. 홈에서 다른 페이지로 이동한다.
  3. 다른 페이지에서 홈으로 왔을 때 배너가 계속 열려 있다.

원하던 상황

배너를 한번 닫으면, 다른 페이지에서 홈으로 왔을 때 배너가 닫혀있도록 합니다.
혹은 쿠키를 이용하여 만료 시간이 지난 후에 다시 배너가 열리도록 합니다.
@inyeong-kang inyeong-kang added this to the 6차 데모데이 milestone Oct 4, 2023
@Gilpop8663
Copy link
Collaborator

구현 내용

  • 배너를 닫았을 경우 1시간동안 안보이도록 쿠키 설정

상세 내용

  • 배너를 닫았을 경우 useContext를 사용하기엔 코드 사용되는 양이 너무 많다고 생각이 들어서 쿠키를 이용해 1시간만 안보이도록 할 예정

Gilpop8663 added a commit that referenced this issue Oct 19, 2023
@Gilpop8663
feat: (#707) 쿠키를 이용해 배너를 닫으면 1일동안 보이지 않도록 구현
51b9796
@Gilpop8663 Gilpop8663 mentioned this issue Oct 19, 2023
Merged
Gilpop8663 added a commit that referenced this issue Oct 19, 2023
@Gilpop8663
공지사항 목록 페이지 / 공지사항 상세 페이지 / 공지사항 배너 데이터 패치하기 / 배너 닫기 쿠키 설정 / dynamic …
9f28b83 
…import 적용 (#787)

* feat: (#785) 공지사항 목록 불러오는 컴포넌트 구현
NoticeList 컴포넌트와 NoticeList 타입 이름이 겹쳐서 타입명을 NoticeListType으로 변경

* fix: (#785) 스켈레톤 width 설정이 되어있지 않아서 안보이는 문제 해결

* feat: (#785) 공지사항 목록 서스펜스, 에러바운더리 설정
패치중일때 더보기 버튼 비활성화

* feat: (#785) 공지사항 상세 페이지 실제 데이터 불러오도록 구현

* fix: (#785) 공지사항 상세 데이터를 불러올 때 NaN으로 불러오던 것 수정

* fix: (#785) 공지사항 수정이 안되던 문제 수정
공지사항 작성, 수정에 성공했을 경우 공지사항 목록 url로 이동하도록 구현

* feat: (#707) 쿠키를 이용해 배너를 닫으면 1일동안 보이지 않도록 구현

* feat: (#785) 배너 공지사항 실제 데이터로 불러오도록 구현
사용하지 않는 기존의 공지 관련된 파일들 삭제

* refactor: (#785) 코드 가독성을 위해 배너 관련된 코드 분리

* feat: (#785) 배너에 사용되는 스켈레톤 구현

* feat: (#785) 배너가 에러가 났을 경우 아무것도 렌더링 되지 않도록 설정

* feat: (#785) 공지사항 생성,수정,삭제에 실패했을 때 토스트로 에러 메세지 출력

* feat: (#785) 배너 공지사항이 없을 경우 null을 보내주는 상황 처리

* feat: (#785) 번들 사이즈 최적화를 위해 관리자 페이지 dynamic import 적용

* chore: (#785) 에러 메세지 문구 수정

* refactor: (#785) BannerErrorBoundary => NoRenderErrorBoundary 이름 변경
@github-project-automation github-project-automation bot moved this from Todo to Done in 2023-votogether Oct 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Gilpop8663 Gilpop8663

Labels
FE 🐛 버그 🚨 우선 순위 높음
Projects
2023-votogether
Status: Done
Milestone
6차 데모데이
2 participants
@Gilpop8663 @inyeong-kang