I think that we should have used more of the Pickup.cs functions like the ClickablePickup.cs does.
I did mine like this.
//public void OnTriggerEnter(Collider other)
//{
// if (pickup.CanBePickedUp())
// {
// pickup.PickupItem();
// }
// else
// {
// Debug.Log(“No”);
// }
//}
because, we should see if there is a space to be had if the item can be picked up or not. I also like the way he did the check for the player, as the tag. but Just a note.