Windows has a hard limit on the number of shared connections. And its not based on the number of users connected but on the number of connections everybody makes. This not only includes the mapped drives but printers as well. I don't know about Windows 10, but for Windows 7 the limit was 20 concurrent connections and earlier versions of windows had less.
But the setup is good if you have few users.
My personal preference when setting this up is to completely disable cache in the shared folder. Because (1) less network consumption and (2) safer version control. You can also set permissions on who has Write/Copy/Execute permissions.