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

Y_FE_Toy1_Team6 - Youcandoit #7

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

Y_FE_Toy1_Team6 - Youcandoit #7

wants to merge 237 commits into from

Conversation

Eojoonhyuk
Copy link

@Eojoonhyuk Eojoonhyuk commented Sep 22, 2023

πŸ§‘πŸ»β€πŸ’» 직원듀을 μœ„ν•œ μœ„ν‚€ μ‚¬μ΄νŠΈ ( ν† μ΄ν”„λ‘œμ νŠΈ1 6μ‘° μœ‘μΊ”λ‘μž‡ )

youcandoit_logo

πŸ’πŸ»β€β™‚οΈ μ•Όλ†€μž ν…Œν¬μŠ€μΏ¨ 1κΈ°, ν† μ΄ν”„λ‘œμ νŠΈ 6쑰의 직원듀을 μœ„ν•œ μœ„ν‚€ μ‚¬μ΄νŠΈ ν”„λ‘œμ νŠΈμž…λ‹ˆλ‹€.

πŸ“Œ ν”„λ‘œμ νŠΈ Info

image

  • πŸ”— ν”„λ‘œμ νŠΈ URL : youcandoit-wiki

  • πŸ—“οΈ 개발 κΈ°κ°„ : 20203.09.08 ~ 2023.09.22

  • πŸ”¨ μ‚¬μš© 기술 μŠ€νƒ

νŒ€μ› μ†Œκ°œ

μ–΄μ€€ν˜(리더) 박지성 μ΄μ—°μˆ˜ μž₯영민 채민석
Eojoonhyuk flamozzi suehub yeongmins chaeminseok
Eojoonhyuk flamozzi suehub yeongmins chaeminseok

βœ” ν•„μˆ˜ κ΅¬ν˜„μ‚¬ν•­

  • λ¬Έμ„œνŽΈμ§‘, revision κΈ°λŠ₯을 제곡(CRUD κ°€λŠ₯ν•˜κ²Œ)ν•˜μ—¬ 업무일지λ₯Ό μž‘μ„±ν•  수 μžˆλŠ” 직원듀을 μœ„ν•œ μœ„ν‚€μ‚¬μ΄νŠΈ κ΅¬ν˜„(λ§ˆν¬λ‹€μš΄ ν˜•μ‹)
  • firebase database (Firestore) 이용
  • λͺ¨λ‹¬μ„ ν™œμš©ν•œ 근무 μ‹œκ°„μ„ ν‘œμ‹œν•˜λŠ” μ‹œκ³„ 및 타이머 μ°½ κ΅¬ν˜„
  • μΊλŸ¬μ…€μ„ ν™œμš©ν•œ νšŒμ‚¬ 곡지 νŽ˜μ΄μ§€
  • 가러리 νŽ˜μ΄μ§€ / 업무일지 νŽ˜μ΄μ§€ λ“± 메뉴λ₯Ό 필터링 λ˜λŠ” μΉ΄ν…Œκ³ λ¦¬ν™” ν•˜λŠ” 선택바 κ΅¬ν˜„
  • netlify 등을 μ΄μš©ν•œ 정적 νŽ˜μ΄μ§€ 배포
  • TypeScript μ‚¬μš© ν•„μˆ˜
  • κ³Όμ œμ— λŒ€ν•œ μ„€λͺ…을 ν¬ν•¨ν•œ README.md 파일 μž‘μ„±
    • νŒ€μ›λ³„λ‘œ κ΅¬ν˜„ν•œ λΆ€λΆ„ μ†Œκ°œ

βœ” 선택 κ΅¬ν˜„μ‚¬ν•­

  • React μ‚¬μš©μ€ 선택
  • 기타 λ™μž‘μ΄ μ™„λ£Œλ˜κΈ° 전에 λ‘œλ”© μ• λ‹ˆλ©”μ΄μ…˜ κ΅¬ν˜„
  • νŽ˜μ΄μ§€λ„€μ΄μ…˜
  • κ΄€λ ¨λœ 기타 κΈ°λŠ₯도 κ³ λ €
  • eslint μ„€μ •, μ»€λ°‹μ»¨λ²€μ…˜, λ¬Έμ„œν™” λ“± νŒ€ν”„λ‘œμ νŠΈμ‹œ ν•„μš”ν•œ μΆ”κ°€ μž‘μ—…λ“€

βœ” νŒ€μ›λ³„ μ£Όλ‹΄λ‹Ή κ΅¬ν˜„λΆ€

νŒ€μ› λͺ¨λ‘ μžμ‹ μ˜ μ£Όλ‹΄λ‹Ή κ΅¬ν˜„λΆ€ 이외에도 λ¦¬νŒ©ν† λ§ 및 λ‹€λ₯Έ νŒ€μ›μ˜ ν…ŒμŠ€ν¬μ— μ°Έμ—¬ν•˜λ©° ν”„λ‘œμ νŠΈ μ „μ²΄μ μœΌλ‘œ ν˜‘μ—…ν•˜μ˜€μŠ΅λ‹ˆλ‹€.

  • μ–΄μ€€ν˜: νŒ€ 리더, ν”„λ‘œμ νŠΈ 관리 및 병합, ν™ˆ νŽ˜μ΄μ§€ λ‹΄λ‹Ή
  • 박지성: μœ„ν‚€ νŽ˜μ΄μ§€ 및 λ§ˆν¬λ‹€μš΄ 에디터 λ‹΄λ‹Ή
  • μ΄μ—°μˆ˜: 가러리 νŽ˜μ΄μ§€ 및 이미지 처리, 둜그인 λ‹΄λ‹Ή
  • μž₯영민: λΌμš°νŒ… 및 μ‚¬μ΄λ“œλ°”, λ””μžμΈ λ‹΄λ‹Ή
  • 채민석: μΆœν‡΄κ·Ό 타이머 및 νšŒμ›κ°€μž… λ‹΄λ‹Ή

✏️ userflow

youcandoit_userflow

πŸ§‘πŸ»β€πŸ’» κ΅¬ν˜„ 사항

πŸ“ ν™ˆ

μΆœν‡΄κ·Ό 타이머

ν™ˆ_μΆœν‡΄κ·Ό

ν™ˆ_μΊλŸ¬μ…€κ³Ό 곡지 및 가러리

ν™ˆ_κ³΅μ§€κ°€λŸ¬λ¦¬

ν™ˆ_μΊ˜λ¦°λ”

ν™ˆ_μΊ˜λ¦°λ”

πŸ“ 둜그인 및 νšŒμ›κ°€μž…

둜그인 및 νšŒμ›κ°€μž…, λΉ„λ°€λ²ˆν˜Έ 찾기와 λ‘œκ·Έμ•„μ›ƒ

둜그인_νšŒμ›κ°€μž…

πŸ“ 곡지

곡지사항 읽기와 μž‘μ„± 및 μ‚­μ œ

곡지

πŸ“ μœ„ν‚€

μ‚¬μ΄λ“œλ°” 탐색과 λ¬Έμ„œ μˆ˜μ • 및 μ €μž₯

μœ„ν‚€

πŸ“ 가러리

가러리 사진 읽기 및 μ‚¬μ΄λ“œλ°” 필터링과 μ—…λ‘œλ“œ 및 μˆ˜μ •κ³Ό μ‚­μ œ

가러리

πŸ“ λ°˜μ‘ν˜• κ΅¬ν˜„

ν™ˆ λ°˜μ‘ν˜• κ΅¬ν˜„

ν™ˆλ°˜μ‘ν˜•

곡지와 μœ„ν‚€ 및 가러리 λ°˜μ‘ν˜• κ΅¬ν˜„

κ³΅μ§€μœ„ν‚€κ°€λŸ¬λ¦¬λ°˜μ‘ν˜•

Eojoonhyuk and others added 30 commits September 10, 2023 19:29
Change to commute button component in header
μ½”λ“œ ν•œμ€„ μ‚­μ œ ν–ˆμŠ΅λ‹ˆλ‹€.
design: SideBar html, css μž‘μ—…
NamgungJongMin added a commit that referenced this pull request Sep 22, 2023
Sinary00 added a commit that referenced this pull request Sep 22, 2023
둜그인 κΈ°λŠ₯을 κ΅¬ν˜„ν•œλ‹€.
im-na0 added a commit that referenced this pull request Sep 22, 2023
Feat: Employee νŽ˜μ΄μ§€ λ§ˆν¬μ—…
TaePoong719 added a commit that referenced this pull request Sep 22, 2023
λ§ˆν¬λ‹€μš΄ 에디터 κ΅¬ν˜„μ„ μœ„ν•œ 초기 μ„ΈνŒ… (이슈 #7)
Copy link
Member

@marshallku marshallku left a comment

Choose a reason for hiding this comment

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

고생 λ§ŽμœΌμ…¨μŠ΅λ‹ˆλ‹€~
κ΅¬ν˜„ν•˜λ €κ³  ν•˜μ…¨λ˜ μ£Όμš”ν•œ κΈ°λŠ₯듀은 λͺ¨λ‘ 잘 κ΅¬ν˜„λœ 것 κ°™λ„€μš”!

λ‹€μŒ ν”„λ‘œμ νŠΈ 전에

  • react λ‚΄μ—μ„œ μƒνƒœ 처리
  • useCallback, useMemo λ“± λ‹€μ–‘ν•œ hooks ν™œμš©
  • λ°˜λ³΅λ˜λŠ” 둜직과 μ»΄ν¬λ„ŒνŠΈ 곡톡화
  • νŒ”λ ˆνŠΈ 등을 ν™œμš©ν•œ μŠ€νƒ€μΌ κ΄€λ ¨ μƒμˆ˜ μž¬μ‚¬μš©
  • cssλ₯Ό ν†΅ν•œ λ ˆμ΄μ•„μ›ƒ ꡬ성

λ₯Ό 쑰금 더 μ‚΄νŽ΄λ³΄μ‹œλ©΄ 쒋을 것 κ°™μŠ΅λ‹ˆλ‹€!

μ€‘λ³΅λ˜λŠ” λ¦¬λ·°λŠ” μƒλž΅ν–ˆμœΌλ‹ˆ, 본인의 μ½”λ“œμ— λŒ€ν•œ 리뷰가 μ•„λ‹ˆλ”λΌλ„ ν•œ λ²ˆμ”© μ°Έκ³ ν•΄μ£Όμ‹œλ©΄ κ°μ‚¬ν•˜κ² μŠ΅λ‹ˆλ‹€~

<meta name="theme-color" content="#000000" />
<meta
name="description"
content="Web site created using create-react-app"
Copy link
Member

Choose a reason for hiding this comment

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

이런 정보듀도 같이 λ°”κΎΈμ‹œλ©΄ 쒋지 μ•Šμ„κΉŒμš”?

Copy link
Member

Choose a reason for hiding this comment

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

PWA둜 λ§Œλ“œλŠ” κ±Έ μ‹œλ„ν•˜μ‹  것 κ°™λ„€μš”!
μ•„μ΄μ½˜κ³Ό μ„œλΉ„μŠ€ μ›Œμ»€λ₯Ό μΆ”κ°€ν•˜μ…”μ„œ PWAλ₯Ό 완성해보셔도 쒋을 것 κ°™μŠ΅λ‹ˆλ‹€.

Comment on lines +1 to +4
@font-face {
font-family: "NotoSansKR-Bold";
src: url("./NotoSans/NotoSansKR-Bold.ttf");
}
Copy link
Member

Choose a reason for hiding this comment

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

woff2 폰트λ₯Ό μ‚¬μš©ν•˜κ³ , font-display:swap을 μΆ”κ°€ν•˜λŠ” λ“± μ›Ήν°νŠΈ μ΅œμ ν™”λ₯Ό 진행해보면 쒋을 것 κ°™μŠ΅λ‹ˆλ‹€.
NotoSansλŠ” Subset font도 λ§Œλ“€μ–΄μ Έμžˆμ„ κ±°μ—μš”.

Comment on lines +2 to +4
import github from "../../assets/icons/footer_icon/footer_github_black_icon.png";
import logo from "../../assets/icons/header_icon/logo.png";
import "../../assets/fonts/Font.css";
Copy link
Member

Choose a reason for hiding this comment

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

aliasλ₯Ό μ‚¬μš©ν•΄λ³΄μ‹œλ©΄ 쑰금 더 κΉ”λ”ν•˜κ²Œ importλ₯Ό 정리할 수 μžˆμ„ 것 κ°™λ„€μš”!

<a
href="https://github.com/Fastcampus-Youcandoit/frontend"
target="_blank"
rel="noreferrer">
Copy link
Member

Choose a reason for hiding this comment

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

noreferrerκΉŒμ§€ μΆ”κ°€ν•˜μ‹  것 μ’‹μŠ΅λ‹ˆλ‹€! noopenerκΉŒμ§€ μΆ”κ°€λ˜λ©΄ 쒋을 것 κ°™λ„€μš”.

Comment on lines +130 to +131
if (location.state.id)
await updateDoc(doc(db, "notice", location.state.id), data);
Copy link
Member

Choose a reason for hiding this comment

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

elseκ°€ μžˆκ±°λ‚˜, if문이 두 쀄이 λ„˜μ–΄κ°€κΈ° μ‹œμž‘ν•˜λ©΄ μ€‘κ΄„ν˜Έλ₯Ό μƒλž΅ν•˜μ§€ μ•ŠλŠ” 게 μœ μ§€λ³΄μˆ˜μ„± λ“±μ˜ λ©΄μ—μ„œ 더 쒋을 것 κ°™μŠ΅λ‹ˆλ‹€!

// 1.이메일 μœ νš¨μ„±κ²€μ‚¬
// 영문과 이메일 ν˜•μ‹
const checkEmailValidation = (value: string) => {
const EMAIL_REGEX = /^[a-zA-Z0-9._%+-]+@[a-zA-Z0-9.-]+.[a-zA-Z]{2,}$/;
Copy link
Member

Choose a reason for hiding this comment

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

validation κ΄€λ ¨ λ‘œμ§μ΄λ‚˜ regexλŠ” λ”°λ‘œ κ΄€λ¦¬ν•˜λ©΄ μž¬μ‚¬μš©μ„±μ΄ 높아지지 μ•Šμ„κΉŒμš”?

Comment on lines +53 to +54
const [email, setEmail] = useState<string | undefined>("");
const [password, setPassword] = useState<string | undefined>("");
Copy link
Member

Choose a reason for hiding this comment

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

두 값이 empty string이 μ•„λ‹ˆλΌ undefinedκ°€ 될 수 μžˆλŠ” κ±΄κ°€μš”?

Comment on lines +95 to +100
let isValidName;
if (value.length === 0) {
isValidName = "required";
} else {
isValidName = NAME_REGEX.test(value) ? true : "invalid";
}
Copy link
Member

Choose a reason for hiding this comment

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

requiredλ‚˜ invalidλŠ” μ–΄λ–€ 역할을 ν•˜λŠ” κ±΄κ°€μš”?
validation λ‘œμ§μ— λΉ„μŠ·ν•œ νŒ¨ν„΄μ΄ λ°˜λ³΅λ˜λŠ”λ°, form의 μœ νš¨μ„±μ„ κ²€μ‚¬ν•˜λŠ” library(예: react hook form)λ₯Ό μΆ”κ°€ν•˜κ±°λ‚˜, custom hook으둜 묢어보면 쒋을 것 κ°™λ„€μš”.

Comment on lines +45 to +52
font-size: 21px;
margin-left: 20px;
font-family: "NotoSansKR-Medium";
`;

const Text = styled.div`
height: 3.5rem;
font-size: 18px;
Copy link
Member

Choose a reason for hiding this comment

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

font-sizeμ—” λŒ€λΆ€λΆ„ rem λ‹¨μœ„κ°€ μ‚¬μš©λ˜μ—ˆλ˜λ°, 가급적 λ‹¨μœ„λ₯Ό ν†΅μΌν•˜λ©΄ 쒋을 것 κ°™μŠ΅λ‹ˆλ‹€.

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.

6 participants