What are you expecting to happen? As far as I can tell all you are doing in update is assigning char1 to the player variable everyframe after scoreboard.Character1 becomes true.
If you want to see a visual representation of this, you should probably be assigning char1 to the sprite property of a UI Image. (But even then you probably only want to do this once, rather than in update).
Try changing “player” to the type SpriteRender (and the get component call, too)
Then, set player.sprite = char1;
Agreed that it would be much better to simply switch it when you change the character (or choose the character – whatever that situation is), as opposed to inside Update()