Fixed Get Current theme issue when customized using dashboard (#47)

LoginRadius · Jul 5, 2021 · 2f65236 · 2f65236
1 parent 4e9a509
commit 2f65236
Show file tree

Hide file tree

Showing 4 changed files with 7 additions and 5 deletions.
diff --git a/api/deployment.go b/api/deployment.go
@@ -96,6 +96,8 @@ func GetPage() (*HostedPageResponse, error) {
 	if err != nil {
 		return nil, err
 	}
+	// This Logic is needed to support the theme customization done using Dashboard.
+	resultResp.Pages[0].Status = strings.ReplaceAll(resultResp.Pages[0].Status, "9", "")
 	return &resultResp, nil
 }
 

diff --git a/cli_home.png b/cli_home.png
diff --git a/cmd/get/theme/theme.go b/cmd/get/theme/theme.go
@@ -22,8 +22,8 @@ func NewThemeCmd() *cobra.Command {
 		Example: heredoc.Doc(`
 			$ lr get theme --all
 			Available Themes:
-			1. Tokyo
-			2. London
+			1. London
+			2. Tokyo
 			3. Helsinki
 
 			$ lr get theme --active 
@@ -43,16 +43,15 @@ func NewThemeCmd() *cobra.Command {
 func themes() error {
 	if *all && !*active {
 		fmt.Println("Available Themes:")
-		fmt.Println("1. Tokyo")
-		fmt.Println("2. London")
+		fmt.Println("1. London")
+		fmt.Println("2. Tokyo")
 		fmt.Println("3. Helsinki")
 	} else if *active && !*all {
 		resp, err := api.GetPage()
 		if err != nil {
 			return err
 		}
 		index := resp.Pages[0].Status
-
 		fmt.Println("Current Theme:", cmdutil.ThemeMap[index])
 	} else {
 		fmt.Println("Use exactly one of the following flags: ")

diff --git a/cmdutil/constant.go b/cmdutil/constant.go
@@ -1,6 +1,7 @@
 package cmdutil
 
 var ThemeMap = map[string]string{
+	"0": "London", // Handled fallback logic to London.
 	"1": "London",
 	"2": "Tokyo",
 	"3": "Helsinki",