Dumping a big script like this is really counter intuitive. I know you’re trying to help but you should either;
post a script and go into detail explaining what it does and why it does that
just explain the concepts he needs to go around to doing the thing he wants to do.
Just posting a big script and giving little to no explanation, is in no way what so ever going to help him better his skills as a programmer.
ok i fixed it, lmk if i can improve it
don’t make the gui handling server sided. whenever you’re picking up a tool, a remote should be sent out to the server by the player, so that the server knows when a player picked it up. what you could do is return some kind of bool value from the server so that the client knows if it picked it up or not, and then the client should handle the gui accordingly
it is perfectly fine to handle gui on the server
you’re just putting extra tension on the server when there is no reason to do that