IContentFinders documentation correction

[hfloyd]

What type of issue is it?

Wrong documentation

What article/section is this about?

https://github.com/umbraco/UmbracoDocs/blob/main/13/umbraco-cms/reference/routing/request-pipeline/icontentfinder.md

https://docs.umbraco.com/umbraco-cms/13.latest/reference/routing/request-pipeline/icontentfinder

Describe the issue

In the section "Adding and removing IContentFinders", a code sample is provided:

builder.CreateUmbracoBuilder()
    .AddBackOffice()
    .AddWebsite()
    .AddDeliveryApi()
    .AddComposers()
    .AddCustomContentFinders()
    .Build();

But .AddCustomContentFinders() is not valid. There is an option for .AddContentFinder<MyContentFinder>() is that what should be listed here?

[sofietoft]

Thanks for the report @hfloyd ! 💪

I can see in the code sample above it, that AddCustomContentFinders() is defined.
So, could it be that you've given it a different name in your setup, and need to call this something different in the Program.cs file? 🤔

If that's not the case, let me know, and I'll forward this to our develoeprs.

[hfloyd]

🤦‍♀️ Of course. The previous code sample defines that group of instructions.

For whatever reason I totally missed that and thought that it was like .AddComposers() - a general Umbraco-based instruction that would magically sweep up all the Content Finders in the codebase. You can close this, then, or maybe consider renaming AddCustomContentFinders() to something more obviously "custom" like AddMyCustomContentFinders()

thanks, @sofietoft !

[sofietoft]

Yes! I think that would be a great idea actually.
I'll look into making this change later today 💪

Thanks @hfloyd ! 😄