Closed BarndoBoi closed 2 months ago
Alright, I went and combed through the lobby code hunting more bugs. I think I've now resolved all of the major ones I'd spotted during testing, though I definitely didn't get all of them. Now with the new commits this PR getting merged will resolve issue #10 and have working multiplayer for both implemented roles. I'll likely sort through the diff for this PR for a few minutes before merging it into main and opening up a new branch to work on. UwU
After reviewing the changes made and the libraries imported, I do indeed think that this PR is ready to merge into main. I won't be squashing this one down so the commit history is preserved in main. In effect, this is a "LGTM 👍" pull request. Any additional problems will get resolved in a later dev branch, okay bb? <3
Holy heck, its taken awhile but I've finally got Mirror, the lobby screen, UI, new textures, and a bunch more that are coming with this update. I'm still suspicious that there are a few bugs lingering in the lobby's code, and players connecting after people are readied up or handling their disconnections are not yet implemented and will go in the next PR since it isn't vital right now. I've tested everything as both the host and the client numerous times, but currently the inputs for the client aren't responding right now, which will have an issue opened here soon. All in all, this first step towards making the game multiplayer has been pretty successful!