Skip to content

[Feature] Introduce scene changer#4

Open
kth1888 wants to merge 3 commits intomainfrom
feat/#BB-48
Open

[Feature] Introduce scene changer#4
kth1888 wants to merge 3 commits intomainfrom
feat/#BB-48

Conversation

@kth1888
Copy link
Member

@kth1888 kth1888 commented Aug 31, 2022

기본 깃 조작 및 신 전환 스크립트(SceneChanger.cs) 업로드 + 커밋 + pr 작업 해봤습니다.

@linear
Copy link

linear bot commented Aug 31, 2022

BB-48 예상되는 신들 더미

  1. 시작화면에 텍스트로 시작화면 띄워주고 버튼 아무거나 누르면 게임 신으로 전환

2-1. [최초 1회] 스토리 신 전환

2-2. [최초 1회] 게임 키 설명 신

~~~~ 게임 진행 ~~~~~

순서 없이 트리거만 가능하게 개발

  • 게임 오버 신
  • 엔딩 신

참고

https://www.notion.so/atralupus/460ccde06a414298a313d1ba8dafb025

@upa-r-upa
Copy link
Member

일단 제대로 된 Git 첫 사용 고생하셨습니다. (Hello, Git! 이라고 외치셔야 하는 국룰...)

코드를 보기 이전에 브랜치 이름과 커밋 규약에 대해서 말씀 드립니다. Waterbed 프로젝트에선 규약이 딱히 없지만.. 시맨틱 규약을 쓰신 것 같아 말씀 드려요.

브랜치 이름의 경우 Linear issue id를 달아주신 건 좋은데, 포스팅을 다시 보시면 아시겠지만 그 뒤에 브랜치를 부를 수 있는 별칭까지 적어주시는 게 좋겠습니다.

그리고 커밋 메시지의 경우.. scope(옵셔널)은 그렇게 특정 파일의 이름을 적는 것이 아닙니다.
정답이랄 것 까지는 없지만 예시를 드리자면, 로그인이나 회원가입 기능 등에 관한 커밋이었다면 feat(auth): ... 와 같이 작성할 수 있겠습니다.

덧붙이자면 PR을 열었다고 해서 알림이 오는건 아니기 때문에 Reviewer에 지원님이나 저를 직접 태그해주시면 좋을 것 같습니다.

Copy link
Member

@Atralupus Atralupus left a comment

Choose a reason for hiding this comment

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

추가된 신들에 대한 설명과 SceneChanger에 대한 설명도 PR 본문에 추가해주세요

@Atralupus
Copy link
Member

그리고 TextMesh Pro 같은 라이브러리 내용이 추가될 땐 PR을 따로 올려주세요.

@Atralupus Atralupus changed the title feat(SceneChanger.cs): Added scene changing script [Feature] Introduce scene changer Sep 15, 2022
Copy link
Member

@Atralupus Atralupus left a comment

Choose a reason for hiding this comment

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

튜토리얼, 게임 키 설명 최초 1회만 실행하도록하는 코드가 보이지 않습니다.

using UnityEngine;
using UnityEngine.SceneManagement;

public class SCTutorial : MonoBehaviour
Copy link
Member

Choose a reason for hiding this comment

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

불필요한 SC prefix는 제거해주세요

Copy link
Member

Choose a reason for hiding this comment

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

Scene마다 scene을 변경하는 object용 스크립트가 있는 것 같은데 파일 하나로 관리하는게 좋아보입니다(다음 PR에서 진행하셔도 됩니다)

@Atralupus
Copy link
Member

리베이스도 한번 더 해야겠네요

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.

3 participants