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

feat: firebase 데이터베이스 CRUD #10

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from
Open

feat: firebase 데이터베이스 CRUD #10

wants to merge 1 commit into from

Conversation

yejinleee
Copy link
Collaborator

변경사항

  • env 파일

    • 파이어베이스 데이터베이스 config 정보를 숨기기 위해 환경변수를 쓰고 싶은데,, 바닐라js에서의 사용 어려움으로 ,,, env.js 파일을 만들었습니다. 슬랙에 보내두었으니까 루트경로에 파일 만들고 나서 실행해보시면 됩니다~!
  • dummy.json (데이터베이스 구조 변경)

    • 저장이 필요한 데이터베이스 내용이, 현재 접속한 유저의 게임별 최고점수 & 각 게임별 우승자 이름과 점수. 이니까 각각 usersData 데이터베이스와 winnerData 데이터베이스로 명명하였고 내부 구조도 바꾸었습니다. (depth 구조에 배열이 감싸져있던것 삭제, 점수 Int형으로 변경)
    • 제가 임의로 바꾼거라 석주님 fetchData 명명이랑 부딪쳤습니다. 다들 보시고 컨펌되면 최종 결정안으로 전체 수정해야할것 같습니다!
  • 데이터베이스에 CRUD 기능 함수 구현해두었고, 확인할 수 있게 임의로 버튼 여러개 만들어두었습니다! 어떻게 바뀌는건지 확인할 수 있게 console.log 찍도록 하였으니 확인해보세욥! 그 외에 필요할 것 같은 CRUD 기능 알려주세요!

    • 데이터베이스로 확인하는게 제 구글 계정으로 들어가야 확인이 가능한 것 같아요! 다음에 화면공유로 보여드릴게요
  • 추가 필요한것

    • 요청에 따라 유저이름정보를 보간법으로 사용하기도 함

Copy link
Member

@ksjdev ksjdev left a comment

Choose a reason for hiding this comment

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

기본 기능 테스트 다 해봤는데 다 잘됩니다! 너무 고생하셨어요!
추후에 모였을 때 이야기 하면서 아래 2개 사항 정해보면 될 것 같네요!

제가 임의로 바꾼거라 석주님 fetchData 명명이랑 부딪쳤습니다. 다들 보시고 컨펌되면 최종 결정안으로 전체 수정해야할것 같습니다!

데이터베이스에 CRUD 기능 함수 구현해두었고, 확인할 수 있게 임의로 버튼 여러개 만들어두었습니다! 어떻게 바뀌는건지 확인할 수 있게 console.log 찍도록 하였으니 확인해보세욥! 그 외에 필요할 것 같은 CRUD 기능 알려주세요!

image

몇 가지 기능 테스트를 해봤는데 저희가 구현하는데에 필요로 하는 기능은 모두 존재한다고 판단됩니다! 그래서 저는 지금 방식대로 해도 괜찮을 것 같은데 모여서 의견 공유하면서 확정적으로 결정하면 좋을 것 같아요~! 😃

@harry7435
Copy link
Collaborator

harry7435 commented Dec 13, 2023

addNewData(`usersData/${newUserName}`, newUserData))
새로운 유저 추가 시 이렇게 보간법으로 사용한다는 의미죠?!
기본적인 CRUD 체크는 콘솔도 확인이 다 되네요!! 데이터베이스 구성하신다고 고생많으셨어요 ㅠㅠ 강의 때문에 확인이 늦었습니다.

Copy link
Collaborator

@nimgnas nimgnas left a comment

Choose a reason for hiding this comment

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

고생하셨습니다!! crud 까지..👍🙇‍♂️

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.

4 participants