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

비밀번호 숨기기 기능 추가 #121

Merged
merged 4 commits into from
Jul 28, 2023
Merged

Conversation

deopa0402
Copy link
Contributor

@deopa0402 deopa0402 commented Jul 28, 2023

파일 비밀번호 작성시 *표시로 변경
버튼을 추가하여 ON OFF 가능

  • 추가
    에러문제 해결
image 스크린샷 2023-07-28 오후 1 36 33

@minpeter minpeter requested a review from ananjaemin July 28, 2023 02:01
@minpeter
Copy link
Member

image
image
기능 정상 동작 확인, 에러 무시하고 배포해도 괜찮은지 확인 부탁 @ananjaemin

@deopa0402
Copy link
Contributor Author

passwordInput backgorund color black
텍스트 입력란 backgorund 컬러 수정

스크린샷 2023-07-28 오후 2 24 47 image

@ananjaemin
Copy link
Member

image image 기능 정상 동작 확인, 에러 무시하고 배포해도 괜찮은지 확인 부탁 @ananjaemin

이거는
package.json 파일에서 scripts 부분에 아래 코드 추가 후
"lint": "eslint . --ext ts,tsx --report-unused-disable-directives --max-warnings 0"

아래 명령어 해주면 해결됩니다.
npm run lint -- --fix or yarn run lint -- --fix

@ananjaemin
Copy link
Member

이대로 PR해도 괜찮을듯?
commit message가 아쉬운데 앞으로는

feat : 새로운 기능에 대한 커밋
fix : 버그 수정에 대한 커밋
build : 빌드 관련 파일 수정에 대한 커밋
chore : 그 외 자잘한 수정에 대한 커밋
ci : CI관련 설정 수정에 대한 커밋
docs : 문서 수정에 대한 커밋
style : 코드 스타일 혹은 포맷 등에 관한 커밋
refactor :  코드 리팩토링에 대한 커밋
test : 테스트 코드 수정에 대한 커밋

이런 식의 massge 규칙으로 commit 바람
git commit 컨벤션 이라 검색하면 많이 나옴

@deopa0402
Copy link
Contributor Author

이대로 PR해도 괜찮을듯? commit message가 아쉬운데 앞으로는

feat : 새로운 기능에 대한 커밋
fix : 버그 수정에 대한 커밋
build : 빌드 관련 파일 수정에 대한 커밋
chore : 그 외 자잘한 수정에 대한 커밋
ci : CI관련 설정 수정에 대한 커밋
docs : 문서 수정에 대한 커밋
style : 코드 스타일 혹은 포맷 등에 관한 커밋
refactor :  코드 리팩토링에 대한 커밋
test : 테스트 코드 수정에 대한 커밋

이런 식의 massge 규칙으로 commit 바람 git commit 컨벤션 이라 검색하면 많이 나옴

참고하겠습니다. ❤️

@minpeter minpeter merged commit 4dba1bd into tempfiles-team:main Jul 28, 2023
@minpeter
Copy link
Member

네~ 머지 해드렸습니다.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants