Make sure your player object has a
collider and is assigned the tag
"Player".
Make sure the coin collider is set
to be a Trigger.
Use the debugger or put a Debug.Log line at the top of the OnTrigger function to make sure it is getting called. If it is, then the test for player tag is failing. If it isn't, then the problem is with getting the trigger to fire.