Skip to content

Conversation

@kenowagb
Copy link
Collaborator

Summary

  • Track starting BST insertion order and numbers the player successfully places during a run (TreeRunTracker + TreeShareConfig).
  • Generate a seemytree viewer QR at song end (TreeQrCodeGenerator / ZXing) and show it on the existing results screen beside performance stats.
  • Works for 1P and 2P — each player gets their own tracker URL on their result panel.

Test plan

  • Play GameScene (1P), place some nodes, finish the song
  • Confirm results screen still shows grade/score/hits and Continue/Quit
  • Confirm QR appears (no Scan text); Console logs viewer URL in editor/dev builds
  • Scan QR → seemytree viewer URL matches starting sample order + successful placements only
  • Finish with zero successful placements → URL still has placed= and QR still shows
  • Failed Place attempts do not appear in placed
  • 2P: each side’s QR reflects that player’s tree
  • Change TreeShareConfig base URL in Inspector and confirm the encoded URL updates

Kenneth Enow and others added 2 commits July 12, 2026 15:26
Introduces three main pieces: TreeRunTracker captures the starting BST insertion order, records numbers the player successfully places, and TreeQrCodeGenerator/TreeResultQrDisplay build a seemytree viewer QR shown beside existing performance stats (1P and 2P).

Co-authored-by: Cursor <cursoragent@cursor.com>
@psukesh psukesh merged commit 9ae9965 into main Jul 19, 2026
kenowagb pushed a commit that referenced this pull request Jul 25, 2026
The previous compile fix stripped TreeRunTracker/TreeShareConfig because those
types were not on the old base. Current main already has QR (#7), so this
commit instead keeps QR wiring, drops the unused ExitTutorialMode path (it
called a missing ResetForNewSession), dedupes QR fields after the rebase
merge, and adds the Tutorial ScriptableObjects folder meta.

Co-authored-by: Cursor <cursoragent@cursor.com>
Sign in to join this conversation on GitHub.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants