I also recommend reading up on what @HackItsGood had mentioned since it also describes another important thing when datastoring, connecting the saving to a BindToClose, so if the server shuts down, it’ll still save the data.
You can also learn about how to make your code use UpdateAsync as it would be a more proper way of saving data, here’s a post to help you understand
Only thing I’d do different in the code in the post HackItsGod provided would be to coroutine the SaveData function when it has to go through all the players in the BindToClose