Skip to content

Comments

Fix/#302: SigninForm 테스트 코드 내 필드명 불일치 수정#194

Merged
juyesu merged 1 commit intodevelopfrom
fix/#302-signinform-sync-fieldname
Aug 4, 2025
Merged

Fix/#302: SigninForm 테스트 코드 내 필드명 불일치 수정#194
juyesu merged 1 commit intodevelopfrom
fix/#302-signinform-sync-fieldname

Conversation

@juyesu
Copy link
Collaborator

@juyesu juyesu commented Jul 28, 2025

변경 사항

SigninFormData 타입 필드명과 테스트 코드 내 사용된 필드명이 불일치해서 에러가 발생하는 문제를 수정했습니다.

Feature/#67: 로그인 폼 이메일 기억하기 추가
해당 PR을 통해 추가된 코드로 보이는데 해당 PR 병합할 당시 및 이후에 빌드 에러로 병합 방지가 동작하지 않은 문제는 확인해봐야 할 것 같습니다.

@juyesu juyesu self-assigned this Jul 28, 2025
@netlify
Copy link

netlify bot commented Jul 28, 2025

Deploy Preview for we-write ready!

Name Link
🔨 Latest commit 122b35c
🔍 Latest deploy log https://app.netlify.com/projects/we-write/deploys/68874b5a867e2a0008eb6461
😎 Deploy Preview https://deploy-preview-194--we-write.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@juyesu juyesu changed the title Fix/#302: 테스트 코드 내 필드명 불일치(isRememberEmail) 수정 Fix/#302: SigninForm 테스트 코드 내 필드명 불일치 수정 Jul 28, 2025
Copy link
Collaborator

@SwimmingRiver SwimmingRiver left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

왜 갑자기 빌드에러가 발견됐을까요?
수정 감사합니다!!

@hoxey2Front
Copy link
Collaborator

수고하셨습니다~!

@hoxey2Front hoxey2Front added the Please-Merge 병합시켜주세요! label Aug 1, 2025
@juyesu juyesu merged commit f5f2848 into develop Aug 4, 2025
5 checks passed
@juyesu juyesu deleted the fix/#302-signinform-sync-fieldname branch August 4, 2025 03:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Please-Merge 병합시켜주세요!

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants