16주차 Assignment - 홍경욱 #4
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
khj0426
reviewed
Nov 14, 2024
| const email = document.getElementById("signupEmail").value.trim(); | ||
| const password = document.getElementById("signUpPassword").value; | ||
|
|
||
| if (!email || !/^[^\s@]+@[^\s@]+\.[^\s@]+$/.test(email)) { |
There was a problem hiding this comment.
요런 특정 조건을 검증하는 친구들
예를 들면 8~16글자의 특수문자를 포함한 비밀번호 형식이나, 지금과 같이 이메일 형식은 컴포넌트와는 분리해서 위치시키는 게 좋아요!
컴포넌트는 최대한 여러 종속적인 친구들과 분리를 하는 게 좋습니다.
말 그대로 ui만 보여주는 역할만 하는게 제일 이상적인 컴포넌트에요
그래서 지금처럼 특정 조건의 검증, api찌르는 함수 등은 다른 폴더나 파일에서 관리해주는 게 바람직해요!
| .catch((error) => { | ||
| const errorCode = error.code; | ||
| const errorMessage = error.message; | ||
| console.error("회원가입 오류:", errorCode, errorMessage); |
There was a problem hiding this comment.
alert를 띄워주면 굳이 콘솔에 찍을 이유가 없습니다.
말그대로 콘솔은 개발자에게만 보여주는 게 바람직하고 사용자에게는 최대한 보여주지 않는게 이상적이에요
| signInWithEmailAndPassword, | ||
| } from "https://www.gstatic.com/firebasejs/11.0.1/firebase-auth.js"; | ||
|
|
||
| const firebaseConfig = { |
There was a problem hiding this comment.
잘 하시겠지만, 요런 환경을 세팅해주는 친구들(파이어베이스, 다른 도구들의 설정 객체들?)은 꼭 환경변수로 주입해주세요
| <button type="button">회원가입</button> | ||
| </form> | ||
|
|
||
| <script type="module"> |
There was a problem hiding this comment.
사실 자바스크립트는 모듈을 얼마나 잘 이해하는지가 중요한 거 같아요.
모듈은 꼭꼭 공부해주세요! 아래와 같은 키워드를 파보면 좋을 거 같아요
- Commons JS VS ES 모듈
- 번들링 과정이 뭐지?
- 모듈과 트리셰이킹
특히 이 트리셰이킹은 실제 개발하실 떄, 성능과도 되게 밀접해요.꼭 공부해주새용
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
fire base를 활용하여 로그인 구현
css 추가하기
Important content
없습니다
Question
없습니다
Reference
강의 자료