episerver / upgrade-assistant-extensions

Apache License 2.0
10 stars 6 forks source link

IFindUIConfiguration #7

Closed lunchin closed 3 years ago

lunchin commented 3 years ago

Replace with FindOptions

modulus37 commented 3 years ago

Hi Mark

Here I have found the following pattern.

in the .NET Framwork Foundation Project: Features/Search/SearchService.cs where it exists
public SeaarchService(ICurrentMarket currentMarket, .....
IFindUIConfiguration findUIConfiguration, ....

in the Net5 Soluiton: it is just comented out. public SearchService(ICurrentMarket currentMarket, ....... //IFindUIConfiguration findUIConfiguration, ...

can this just be removed?
this also would eliminate this type: using EPiServer.Find.UI;

correct?

modulus37 commented 3 years ago

Hi Mark can you please add more details on this. Many thanks René