Remote Event not firing?

[Ukendio (Marcus)]

You haven’t used the remoteEvent at all, you only defined it.
For more defensive scripting, try using a more signal-styled coding pattern.

Avoiding wait() and why

This is an adaptation of a Twitter thread I made a while ago. This is simply a better formatted and more fleshed out version. Let’s talk about wait(). Not wait(n) (although if you are using small n values to where it’d be barely distinguishable from wait(), you’re probably going to hit the same issues mentioned in this thread). We probably learned about wait() at first as the solution to “why are my infinite loops crashing me?”. It’s a super easy solution and it works! This starts a bad habit…

Wait function alternative?

Though admittedly better, :Wait() is still unreliable and may defer. A coroutine-based solution where you explicitly resume the thread is likely to be more reliable/consistent. This solution is fine as is, but :Wait() does still have issues. Here’s an explanation written up by @evaera that does a better job explaining this than I can: (not my screenshot) [image|100%]

Currently you’re querying for a player to join instead of listening to their joins.