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

Feature/testplayer #14

Merged
merged 15 commits into from
Nov 21, 2024
Merged

Feature/testplayer #14

merged 15 commits into from
Nov 21, 2024

Conversation

jihu0331
Copy link
Contributor

"Add: State transitions, Interactions with props"

@jihu0331
Copy link
Contributor Author

Add: State transitions, Interactions with props

"chore: Class hierarchy change"

team-project-for-2024-fall-swpp-team-03 Outdated Show resolved Hide resolved
ProjectSettings/ProjectSettings.asset Outdated Show resolved Hide resolved
Assets/New Physic Material.physicMaterial Outdated Show resolved Hide resolved
@jihu0331
Copy link
Contributor Author

"chore:review reflected"

@@ -52,7 +53,7 @@ private void Awake()
_collider = GetComponent<Collider>();
_physicMaterial = _collider.material;
_isHoldingJump = false;
_isGrounded = 0;
_isGrounded = false;
Copy link
Contributor

Choose a reason for hiding this comment

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

IsGrounded 라는 프로퍼티로 만드는건 어떨까 싶습니다

@@ -174,18 +160,23 @@ public void InteractWithProp(PropBase prop)

private void OnCollisionEnter(Collision collision)
{

var contactPoint = collision.contacts[0];
if (contactPoint.normal.y > 0.7f)
Copy link
Contributor

Choose a reason for hiding this comment

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

조정이 편하게 에셋으로 빼는건 어떨까요

@integraldx integraldx merged commit d26c60c into main Nov 21, 2024
4 checks passed
@integraldx integraldx deleted the feature/testplayer branch November 21, 2024 13:32
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