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: Custom Rules for Direct & Teams #74

Open
wants to merge 12 commits into
base: master
Choose a base branch
from

Conversation

rapito
Copy link
Contributor

@rapito rapito commented May 9, 2021

Features

Adds ability to create Custom Ruleset Games on Direct and Teams Modes

  • Added Custom Rules Button
  • Initialized Custom Rules text based on ruleset (disappears when connected and shows "playing {opponent}")
  • Added a Quick Reset button to restore Default Ruleset (Press Z on Custom Rules Menu)
  • Adds in-game Text to indicate if current game has Custom Rules.
  • Random Stage is obtained from the game if on Direct/Teams and Custom Rules is set instead of Dolphin.

Changes

  • Send current random stages settings along with Match Selections to be able to sync between games.
  • Send Match Info to Dolphin after the game has built it using local rules (InitOnlinePlay.asm).
  • Stall before retrieving Match Info from Dolphin until all players are ready (InitOnlinePlay.asm).
  • Sync random stages settings for next match (InitOnlinePlay.asm)

Preview

in-game text

image

Reset Ruleset Button (might change it to make it look similar to UI's on name Entry)

image

Working Demo Video

pe.mov

@rapito
Copy link
Contributor Author

rapito commented Sep 8, 2021

  • Merged with upstream
  • Added changes requested by Fizzi so only the decider can set custom rules.

… in state

+ Refactor to be able to reset custom rules from anywhere using compute branch target address macro
@rapito
Copy link
Contributor Author

rapito commented Sep 9, 2021

  • Reset Ruleset back to tournament when users disconnect

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.

1 participant