Challenge 1 / 1

We made UserRepository implement UserLoaderInterface. Why did we also have to delete property: email from app_user_provider?

property has to go whenever the repository implements any extra interface.

property only applies to in-memory providers, so it was dead config anyway.

With property set, the entity provider just queries that one field - removing it is what makes it call our loadUserByIdentifier().

Leaving it would run both the property lookup and the loader, and Symfony would find two users.