Skip to content

Feat net/state #34

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
Open
wants to merge 9 commits into
base: main
Choose a base branch
from
Open

Feat net/state #34

wants to merge 9 commits into from

Conversation

ram02z
Copy link
Owner

@ram02z ram02z commented Jan 12, 2023 •
edited
Loading

Description

  • Added 6 interactable cubes
  • Added interactable point to tmpcharacter
  • Every interactable is assigned a question and a code
  • Updated quizcanvas to support changes

Issues

  • If multiple questions are on cooldown, the countdown text gets overlapped
  • Points are shared
  • Interactables should be created at runtime
  • Interaction with objects when another player exists in game is buggy
    • Questions are different (need to be synced)
    • Codes are different (need to be synced)
    • If one character is in interaction range and another character is not, the other character can still open the quiz

Linked issue(s)
Closes #31

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Documentation update (non-breaking change which updates documentation)

Checklist

  • I have performed a self-review of my code
  • My code follows the style guidelines of this project
  • New and existing unit tests pass locally with my changes

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@alexshaul alexshaul Awaiting requested review from alexshaul alexshaul is a code owner

@Jahmii Jahmii Awaiting requested review from Jahmii Jahmii is a code owner

@psykl5 psykl5 Awaiting requested review from psykl5 psykl5 is a code owner

@RoniHab RoniHab Awaiting requested review from RoniHab RoniHab is a code owner

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Global state for multiplayer game
1 participant
@ram02z