search

OrchardCMS / OrchardCore

Orchard Core is an open-source modular and multi-tenant application framework built with ASP.NET Core, and a content management system (CMS) built on top of that framework.
https://orchardcore.net
BSD 3-Clause "New" or "Revised" License
7.41k stars 2.39k forks source link

Dev branch: Elasticsearch index, Sync with Lucene button issue #13495

Open Skrypt opened 1 year ago

Skrypt commented 1 year ago

The button works only once. The second time it throws an exception.

hishamco commented 1 year ago

Seems a few Elasticsearch PRs have been merged recently

/cc @MikeAlhayek

MikeAlhayek commented 1 year ago

@Skrypt There was a change I was planning to do in the previous PR but did not, maybe you can include it in your PR

https://github.com/OrchardCMS/OrchardCore/blob/main/src/OrchardCore.Modules/OrchardCore.Search.Elasticsearch/Controllers/AdminController.cs#L199

All the alert error and notification messages should use the index name as it appears on the UI not the index full name. Each tenant should not have to worry about the full name of index on the server {global_prefix}_{tenant_name}_{ui_index_name} We should only make visible the ui_index_name as we do in the Reset action message.

sebastienros commented 1 year ago

Is this still an issue?