New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Registration of populator uses indexname #22

Open

bielu opened this issue Jan 3, 2021 · 0 comments

bielu commented Jan 3, 2021

I have issue with that line

UmbracoExamine.PDF/src/UmbracoExamine.PDF/PdfIndexPopulator.cs

Line 46 in d98815a

RegisterIndex(PdfIndexConstants.PdfIndexName);

, as based on Elastic Requirements i had to lower cases all index names and I am hitting issue with registering populator for elastic provider. Why wasn't used there some interface to flag index in similar way to Umbraco indexes:
https://github.com/umbraco/Umbraco-CMS/blob/e220baf00ca0a346f3ee17964f14bc143734094f/src/Umbraco.Examine/ContentIndexPopulator.cs#L17
?

bielu mentioned this issue

introduce flag interface to allow to not check by names #23

Open

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment