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

fix: 프로젝트 검색값이 비어있을시 파라미터 제거 및 empty 뷰 추가 #1507

Closed
wants to merge 4 commits into from

Conversation

simeunseo
Copy link
Member

🤫 쉿, 나한테만 말해줘요. 이슈넘버

🧐 어떤 것을 변경했어요~?

  • 검색 파라미터의 value가 비어있다면 파라미터 key,value를 아예 제거합니다.
  • 검색 결과값이 없을 때 보여줄 empty 뷰를 추가합니다.

🤔 그렇다면, 어떻게 구현했어요~?

  • 디바운싱에 넘겨주는 검색값이 빈 문자열이라면, undefined을 넘겨주어 해당 쿼리파라미터(?name=)를 삭제합니다
  • '이용가능한 서비스', '창업 중' 필터링 뱃지에 대해서도 마찬가지로, 필터링 값이 false라면 undefined를 넘겨주어 해당 쿼리파라미터(?isAvailable=, ?isFounding=)를 삭제합니다.
  • 검색 결과가 없을 시 '멤버' 뷰에서 사용하는 empty 뷰와 동일하게 보여줍니다.

❤️‍🔥 당신이 생각하는 PR포인트, 내겐 매력포인트.

결과값 없을 시 empty 뷰 관련

프로젝트 탭에서 결과값이 없을 때 어떻게 보여주는지에 대한 내용을 지난 기수 히스토리에서 찾지 못하여... 멤버탭과 동일하게 넣게 되었습니다. 멤버탭과 동일하게 넣는것이 기존의 없었던 뷰를 대체하는 걸로는 충분하다고 생각해서 이렇게 작업하였습니다!

📸 스크린샷, 없으면 이것 참,, 섭섭한데요?

2024-08-13.9.09.05.mov

url에 주목해서 보시면, 필터링 뱃지를 클릭했다가 해제하면 쿼리 파라미터의 value가 0인채로 남아있으며, 이름을 검색했다가 모두 지웠을 때는 쿼리 파라미터의 value가 빈 문자열로 남아있어서 아무것도 결과값이 뜨지 않는 문제가 발생합니다. 결과값이 없을 때의 예외 뷰도 없습니다.

2024-08-13.9.48.02.mov

필터링 뱃지를 클릭했다가 해제했을 때는 해당 쿼리 파라미터(?isFounding=, ?isAvailable=)를 굳이 url에 가지고있을 필요가 없으므로 아예 삭제하는 걸 볼 수 있습니다.
이름 검색도 마찬가지로 빈 문자열일 경우에는 해당 쿼리 파라미터(?name=)를 제거합니다.
결과값이 없을 때는 empty 뷰를 추가했으며 반응형을 고려했습니다.

@simeunseo simeunseo added the 🐛 Bug 버그픽스 label Aug 13, 2024
@simeunseo simeunseo self-assigned this Aug 13, 2024
Copy link

height bot commented Aug 13, 2024

Link Height tasks by mentioning a task ID in the pull request title or commit messages, or description and comments with the keyword link (e.g. "Link T-123").

💡Tip: You can also use "Close T-X" to automatically close a task when the pull request is merged.

Copy link

🚀 프리뷰 배포 확인하기 🚀

https://3d86a791.sopt-internal-dev.pages.dev

Copy link

✨✨ 스토리북으로 확인하기 ✨✨

@simeunseo simeunseo closed this Aug 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛 Bug 버그픽스 size/L
Projects
None yet
Development

Successfully merging this pull request may close these issues.

fix: 프로젝트 검색값이 비어있을시 파라미터 제거
1 participant