-
Notifications
You must be signed in to change notification settings - Fork 1
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
[FE] 관심사 기능 구현 #24
Comments
어려워 보이네요..! 화이팅!! |
어려울거라고 하시더라구요.. 화이팅입니다 |
sungik-choi
added a commit
that referenced
this issue
Mar 26, 2020
- chip을 set으로 설정, 객체로 관리하려고 한다. - 에러메세지 상수파일로 분리. - 현재 전체적으로 유효성 검증 확인을 errormessage가 null 인거로 파악하고 있는데, 이유는 내 코드에서 에러메세지를 검증하고, 에러메세지가 없으면 null을 반환하도록 했기 때문. - 에러메세지가 없으면 true를 반환하는 메서드를 만들까, 하다가 일을 두번하는 거 같아서 만들지 않았다. 조건문 분기를 2번 돌거 같아서. (에러메시지 생성할 때, 다시 통과여부 판별할 때) - 에러메세지가 존재하지 않는 name에도 에러메세지를 검증하는 메서드가 있는데, 객체를 돌며 반복문으로 실행시키기 위해서... 맞는 방법인지는 모르겠다. - null 을 반환하니 너무 의미가 흐려지는거같아서, 상수에 PASS 라는 값을 null로 등록시켜서, PASS를 반환시키도록 했다(..) - 유효성 스크립트파일에 isValidatePassed 메서드를 추가해서, 통과했다는걸 더 명확히 보여주려고 했다.
sungik-choi
added a commit
that referenced
this issue
Mar 26, 2020
- 비동기 테스트 위해 번들링 파일 우선 하나로. - 바벨-폴리필에서 에러가 발생한다. async await가 제대로 먹지 않아 설치했는데, 이걸 설치하면 여러개 번들링이 안된다.
sungik-choi
added a commit
that referenced
this issue
Mar 27, 2020
- Close #31 - 중복이 많지만 change 이벤트에 중복 확인 코드만 따로 등록하고, 프라미스.then 으로 해결했다. - 코드 스타일 약간 수정. 구조 분해 활용하기 위해서 - async 사용 안했으므로 웹팩 다시 초기화
sungik-choi
added a commit
that referenced
this issue
Mar 27, 2020
- Set 을 사용해 관심사 태그가 생기고 지워지는 기능 구현 - focus 별도로 구현
sungik-choi
added a commit
that referenced
this issue
Mar 27, 2020
- 관심사 태그 실행 속도 보장을 위해, validateInputForms 를 비동기콜백 딜레이 0 으로 구현. - 관심사 입력창 포커스 구현: 메인 파일에 일관성을 떨어트리는 거 같아 별개 파일로 분리. - 칩 생성 파일명 좀 더 포괄적으로 변경
sungik-choi
added a commit
that referenced
this issue
Mar 27, 2020
- 관심사 태그 실행 속도 보장을 위해, validateInputForms 를 비동기콜백 딜레이 0 으로 구현. - 관심사 입력창 포커스 구현: 메인 파일에 일관성을 떨어트리는 거 같아 별개 파일로 분리. - 칩 생성 파일명 좀 더 포괄적으로 변경
beginin15
pushed a commit
that referenced
this issue
Mar 27, 2020
- chip을 set으로 설정, 객체로 관리하려고 한다. - 에러메세지 상수파일로 분리. - 현재 전체적으로 유효성 검증 확인을 errormessage가 null 인거로 파악하고 있는데, 이유는 내 코드에서 에러메세지를 검증하고, 에러메세지가 없으면 null을 반환하도록 했기 때문. - 에러메세지가 없으면 true를 반환하는 메서드를 만들까, 하다가 일을 두번하는 거 같아서 만들지 않았다. 조건문 분기를 2번 돌거 같아서. (에러메시지 생성할 때, 다시 통과여부 판별할 때) - 에러메세지가 존재하지 않는 name에도 에러메세지를 검증하는 메서드가 있는데, 객체를 돌며 반복문으로 실행시키기 위해서... 맞는 방법인지는 모르겠다. - null 을 반환하니 너무 의미가 흐려지는거같아서, 상수에 PASS 라는 값을 null로 등록시켜서, PASS를 반환시키도록 했다(..) - 유효성 스크립트파일에 isValidatePassed 메서드를 추가해서, 통과했다는걸 더 명확히 보여주려고 했다.
beginin15
pushed a commit
that referenced
this issue
Mar 27, 2020
- 비동기 테스트 위해 번들링 파일 우선 하나로. - 바벨-폴리필에서 에러가 발생한다. async await가 제대로 먹지 않아 설치했는데, 이걸 설치하면 여러개 번들링이 안된다.
beginin15
pushed a commit
that referenced
this issue
Mar 27, 2020
- Close #31 - 중복이 많지만 change 이벤트에 중복 확인 코드만 따로 등록하고, 프라미스.then 으로 해결했다. - 코드 스타일 약간 수정. 구조 분해 활용하기 위해서 - async 사용 안했으므로 웹팩 다시 초기화
beginin15
pushed a commit
that referenced
this issue
Mar 27, 2020
- Set 을 사용해 관심사 태그가 생기고 지워지는 기능 구현 - focus 별도로 구현
beginin15
pushed a commit
that referenced
this issue
Mar 27, 2020
- 관심사 태그 실행 속도 보장을 위해, validateInputForms 를 비동기콜백 딜레이 0 으로 구현. - 관심사 입력창 포커스 구현: 메인 파일에 일관성을 떨어트리는 거 같아 별개 파일로 분리. - 칩 생성 파일명 좀 더 포괄적으로 변경
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
관심사
기능
메세지
The text was updated successfully, but these errors were encountered: