From c24dc3112ba0513e55e302fe2d37ba609c610dde Mon Sep 17 00:00:00 2001 From: Richard Wei <54336863+WaysonWei@users.noreply.github.com> Date: Tue, 30 Nov 2021 11:03:08 +1300 Subject: [PATCH] fix(registry): fix order of registries in drop down menu EE-1939 (#5960) Co-authored-by: Prabhat Khera --- .../imageRegistry/por-image-registry.controller.js | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/app/docker/components/imageRegistry/por-image-registry.controller.js b/app/docker/components/imageRegistry/por-image-registry.controller.js index 3e07c77bc..5bbaa0ece 100644 --- a/app/docker/components/imageRegistry/por-image-registry.controller.js +++ b/app/docker/components/imageRegistry/por-image-registry.controller.js @@ -80,10 +80,14 @@ class porImageRegistryController { let showDefaultRegistry = false; this.registries = await this.EndpointService.registries(this.endpoint.Id, this.namespace); + // Sort the registries by Name + this.registries.sort((a, b) => a.Name.localeCompare(b.Name)); + // hide default(anonymous) dockerhub registry if user has an authenticated one if (!this.registries.some((registry) => registry.Type === RegistryTypes.DOCKERHUB)) { showDefaultRegistry = true; - this.registries.push(this.defaultRegistry); + // Add dockerhub on top + this.registries.splice(0, 0, this.defaultRegistry); } const id = this.model.Registry.Id;