nextcloud / context_chat

Chat with your documents
GNU Affero General Public License v3.0
6 stars 0 forks source link

change provider display sequence #38

Closed Rello closed 1 month ago

Rello commented 1 month ago

The listing of the providers should be App->Provider in case several providers exist per app. Currently its the other way round. or is there a reason?

Bildschirmfoto 2024-06-17 um 09 19 25

https://github.com/nextcloud/context_chat/blob/2881badb4e5c32628ebe0d2a16ee459f179154f2/lib/Service/ProviderMetadataService.php#L77

kyteinsky commented 1 month ago

My thinking was the same that it would be better to have the provider names first since that would be different and easily spotted when having multiple providers for one app. I don't have a strong opinion there and app name first would also make sense since the icon would be beside that.