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

16주차 Assignment -서연진 #6

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

shtfh
Copy link

@shtfh shtfh commented Nov 11, 2024

Description

<!— 구현 및 작업 내용을 적어주세요(지우고 작성해주세요) —>

  • ui를 핑크하게 꾸며봤습니다

Important content

<!— 주의 깊게 봐줬으면 하는 부분을 적어주세요(지우고 작성해주세요) —>

  • 핑크

Question

<!— 궁금한 점을 적어주세요(지우고 작성해주세요) —>

  • 없어용

Reference

<!— 참고한 레퍼런스가 있다면 공유해 주세요(지우고 작성해주세요) —>

Copy link

Choose a reason for hiding this comment

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

css파일이 public에 위치해야 하는 이유가 있을까요!!! public폴더에는 보통 어떤 용도로 사용할까요~!

});
});

function toggleForms() {
Copy link

Choose a reason for hiding this comment

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

display와 opacity:0의 차이에 대해 공부해보셔도 좋을 거 같아용

import { getAnalytics } from "https://www.gstatic.com/firebasejs/11.0.1/firebase-analytics.js";
import { getAuth, createUserWithEmailAndPassword, signInWithEmailAndPassword } from "https://www.gstatic.com/firebasejs/11.0.1/firebase-auth.js";

const firebaseConfig = {
Copy link

Choose a reason for hiding this comment

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

요런 환경변수는 리액트를 쓴다면 꼭 env로 주입해주세요

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.

2 participants