Fix PR comment

Author: maartenba

src/content/docs/identityserver/ui/login/dynamicproviders.md

@@ -233,14 +233,14 @@ builder.Services.ConfigureOptions<CustomConfig>();
 ```
 
 :::note
-In your `IConfigureNamedOptions<OpenIdConnectOptions>`, you can use constructor injection to access other services.
-Beware of performance, for example when accessing your Entity Framework Core database context here.
-We recommend caching any additional data you may need as part of configuring options.
-
-Note that the `OidcProvider` class has a `Properties` bag that can be used to store additional dynamic identity provider
-configuration data you could use to further customize the `OpenIdConnectOptions`. When you require this (or other) data
-from the `IIdentityProvider` store, you can use the `ConfigureAuthenticationOptions<>` base class to further customize 
-your dynamic identity provider, as we'll see in the next section.
+While you can use constructor injection in your `IConfigureNamedOptions<OpenIdConnectOptions>` to access other services,
+we recommend using that technique sparingly for performance reasons. If you do need to access your Entity Framework Core
+database context here, make sure to cache any additional data you may need as part of configuring options.
+
+Alternatively, the `OidcProvider` class has a `Properties` bag that can be used to store additional dynamic identity provider
+configuration data. You can use this data to further customize the `OpenIdConnectOptions`, instead of making additional database
+calls. When you require access to the `Properties` bag or the `IIdentityProvider` store, you can use the `ConfigureAuthenticationOptions<>`
+base class to further customize your dynamic identity provider, as we'll see in the next section.
 :::
 
 ### Accessing OidcProvider Data In IConfigureNamedOptions