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.

Sign up for GitHub

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

Jump to bottom

[Fix] 일시정지 관련 버그 일부 해결 #165

Merged
merged 12 commits into from
Nov 28, 2024
Merged

[Fix] 일시정지 관련 버그 일부 해결 #165

merged 12 commits into from
Nov 28, 2024

Conversation

dohyeondol1
Copy link
Contributor

작업 내용

  • 게임이 클리어 되었을 때 일시정지가 가능한 버그를 해결하였씁니다.
  • 플레이어가 이동하는 상태에서 일시정지하면, 이동하는 방향 그대로 전진하는 문제를 해결하였씁니땄!!!!!!!!

참고 사항

  • 18:00 배포를 위해 PR을 먼저 쓰게 되었습니더... 작업은 계속 할게요잇
  • 작업이 다 끝난 시점에 제대로 PR 한번 더 올릴 예정입니다.
  • 리뷰 좀 남겨주이소~

acastracity and others added 10 commits November 27, 2024 03:23
@acastracity
[Fix] 스테이지(감옥) 클리어 이후에도 esc키로 일시정지 창을 띄울 수 있던 문제 해결
8106cbd 
- GoalZoneScript에서 goalZone.stageClear가 false일 때만 일시정지 창 띄우는 함수가 동작하도록 했습니다.
- 원인을 알 수 없지만, PrisonScene -> goalZone object -> Goal Zone Script component -> Stage Clear 란이 '체크(True)'되어 있었습니다.. 이것을 전혀 의심하지 못해 시간이 상당 소요되었습니다.
@acastracity
[Fix] 스테이지(감옥) 클리어 이후에도 esc키로 일시정지 창을 띄울 수 있던 문제 해결
92d0204 
- GoalZoneScript에서 goalZone.stageClear가 false일 때만 일시정지 창 띄우는 함수가 동작하도록 했습니다.
- 원인을 알 수 없지만, PrisonScene -> goalZone object -> Goal Zone Script component -> Stage Clear 란이 '체크(True)'되어 있었습니다.. 이것을 전혀 의심하지 못해 시간이 상당 소요되었습니다.
@acastracity
Merge branch 'develop' into fix/#160/acastracity
0cab19a
@dohyeondol1
Merge branch 'fix/#160/acastracity' of https://github.com/pknu-wap/Ov…
14e7bde 
…erClean into fix/#160/acastracity
@dohyeondol1
[Fix] PrisonUIManager 컴포넌트 RoomUIManager에서 삭제
8ad8895 
- 왜 달았지..? 정말 이유를 모르겠습니다.
@acastracity
Merge branch 'develop' into test/#997/acastracity
a544e66
@acastracity
[Fix] 스테이지(감옥) 클리어 이후에도 esc키로 일시정지 창을 띄울 수 있던 문제 해결
fd0d6d6 
- GoalZoneScript에서 goalZone.stageClear가 false일 때만 일시정지 창 띄우는 함수가 동작하도록 했습니다.
- 원인을 알 수 없지만, PrisonScene -> goalZone object -> Goal Zone Script component -> Stage Clear 란이 '체크(True)'되어 있었습니다.. 이것을 전혀 의심하지 못해 시간이 상당 소요되었습니다.
@acastracity
[Fix] 플레이어가 이동하는 상태에서 일시정지하면, 이동하는 방향 그대로 전진하는 문제 해결
860b1c1 
- 플레이어가 방향 키를 누른 상태에서 esc 버튼을 누르면, 일시 정지 상태에서도 해당 방향으로 이동하던 문제를 해결했습니다.
- 일시 정지 상태에서 이동 속도를 0으로 조정했습니다.
- 애니메이션은 속도가 0인 것처럼 표현하도록 하였으나, 일시 정지 상태에서는 화면이 가려지므로 당장 의미는 없습니다.
@dohyeondol1
Merge branch 'fix/#160/acastracity' of https://github.com/pknu-wap/Ov…
b5b07f2 
…erClean into fix/#160/acastracity
@acastracity
[Fix] 에러 해결
729725e 
- 닫는 괄호 하나를 빼먹어 에러 난 것을 수정했습니다.
@dohyeondol1 dohyeondol1 linked an issue Nov 28, 2024 that may be closed by this pull request
[Fix] 일시정지 관련 버그 해결 #160
Closed
5 tasks
@dohyeondol1 dohyeondol1 changed the title Fix/#160/acastracity [FIx] 일시정지 관련 버그 일부 해결 Nov 28, 2024
@dohyeondol1 dohyeondol1 added 👾버그잡기👾 버그를 고쳐요 🌟리뷰해주세요🌟 리뷰가 필요합니다 🧐도현석🧐 네트워크 기능 구현을 담당합니다 🤓박준우🤓 퍼즐 구현을 담당합니다 ▶️ 진행 중 진행 중 입니다 labels Nov 28, 2024
hyeokbini
hyeokbini approved these changes Nov 28, 2024
View reviewed changes
Copy link
Collaborator

@hyeokbini hyeokbini left a comment

Choose a reason for hiding this comment

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

C3:승인승인 고생했스

acastracity
acastracity approved these changes Nov 28, 2024
View reviewed changes
Copy link
Collaborator

@acastracity acastracity left a comment

Choose a reason for hiding this comment

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

C3:좋았어~~ 화이팅!!

Jinu219
Jinu219 approved these changes Nov 28, 2024
View reviewed changes
Copy link
Collaborator

@Jinu219 Jinu219 left a comment

Choose a reason for hiding this comment

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

C3:승인해요

@Jinu219 Jinu219 changed the title [FIx] 일시정지 관련 버그 일부 해결 [Fix] 일시정지 관련 버그 일부 해결 Nov 28, 2024
@acastracity
Copy link
Collaborator

!! Merge를 위해서 Update From Develop을 하니까 또다른 버그들이 생겨났습니다.. 일단 머지한 이후 새로 이슈파서 해결해보겠습니다

@acastracity acastracity merged commit f58d720 into develop Nov 28, 2024
@acastracity acastracity deleted the fix/#160/acastracity branch November 28, 2024 15:09
@acastracity acastracity added ✅ 완료 완료했습니다! and removed ▶️ 진행 중 진행 중 입니다 labels Nov 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hyeokbini hyeokbini hyeokbini approved these changes

@acastracity acastracity acastracity approved these changes

@Jinu219 Jinu219 Jinu219 approved these changes

Assignees

@dohyeondol1 dohyeondol1

@acastracity acastracity

Labels
✅ 완료 완료했습니다! 🌟리뷰해주세요🌟 리뷰가 필요합니다 👾버그잡기👾 버그를 고쳐요 🤓박준우🤓 퍼즐 구현을 담당합니다 🧐도현석🧐 네트워크 기능 구현을 담당합니다
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[Fix] 일시정지 관련 버그 해결
4 participants
@dohyeondol1 @acastracity @Jinu219 @hyeokbini