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

[Bug] 프로필 수정 페이지 빈 닉네임 허용 x #139

Merged
merged 2 commits into from
Sep 9, 2024

Conversation

wonjichoe
Copy link
Contributor

@wonjichoe wonjichoe commented Sep 9, 2024

🚀 풀 리퀘스트 제안

@coderabbitai: i🤷‍♀️gnore

📋 작업 내용

프로필 수정 페이지 빈 닉네임 허용하지 않게 input 비어 있으면 버튼 비활성화 되게 했습니다.

🔧 변경 사항

  • 📃 README.md
  • 📦 package.json
  • 🔥 파일 삭제
  • 🧹 그 외 ex) .gitignore 등

주요 변경 사항을 요약해 주세요.

📸 스크린샷 (선택 사항)

수정된 화면 또는 기능을 시연할 수 있는 스크린샷을 첨부해 주세요.

📄 기타

추가적으로 전달하고 싶은 내용이나 특별한 요구 사항이 있으면 작성해 주세요.

Summary by CodeRabbit

### Release Notes

#### New Feature
- 프로필 수정 페이지에서 빈 닉네임을 허용하지 않도록 수정되었습니다.
- Loading 텍스트 대신 Spinner 컴포넌트가 추가되었습니다.

#### Refactor
- 수정하기 버튼의 활성/비활성 로직이 변경되었습니다.

#### Style
- `PlaylistInfo` 모듈의 마지막에 줄 바꿈 문자가 추가되었습니다.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
@wonjichoe wonjichoe added the bug 버그가 발생했나요? Something isn't working label Sep 9, 2024
@wonjichoe wonjichoe self-assigned this Sep 9, 2024
Copy link

github-actions bot commented Sep 9, 2024

Image description CodeRabbit

Walkthrough

이번 변경 사항은 PlaylistInfo 모듈의 코드 일관성을 유지하기 위한 줄 바꿈 추가와 프로필 수정 페이지에서 빈 닉네임을 허용하지 않도록 하는 로직 개선, 그리고 사용자 경험을 향상시키기 위해 로딩 텍스트 대신 Spinner 컴포넌트를 도입한 것입니다.

Changes

파일 경로 요약
src/components/playlistDetail/PlaylistInfo.tsx PlaylistInfo 모듈의 마지막에 줄 바꿈 문자가 추가되었습니다.
src/pages/ProfileEdit.tsx 프로필 수정 페이지에서 빈 닉네임을 허용하지 않도록 수정되었고, Loading 텍스트 대신 Spinner 컴포넌트가 추가되었습니다. 또한, 수정하기 버튼의 활성/비활성 로직이 변경되었습니다.

작은 변화, 큰 차이,
코드 속에 숨겨진 마법.
🐇✨ 스피너 돌며, 닉네임 고쳐,
더 나은 세상 만들어가요! 🌟


Uplevel your code reviews with CodeRabbit Pro

CodeRabbit Pro

If you like this project, please support us by purchasing the Pro version. The Pro version has advanced context, superior noise reduction and several proprietary improvements compared to the open source version. Moreover, CodeRabbit Pro is free for open source projects.

Copy link

github-actions bot commented Sep 9, 2024

Visit the preview URL for this PR (updated for commit 65bb48b):

https://toy-project-2-team-4-pizza--preview-cuf8b9eo.web.app

(expires Tue, 10 Sep 2024 05:09:52 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: 70fe82c823fc0ae3a244966b629012f11f284e3b

@devdeun devdeun merged commit 640189a into develop Sep 9, 2024
3 checks passed
@wonjichoe wonjichoe deleted the bug/profile-edit branch September 10, 2024 08:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 버그가 발생했나요? Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants