Add browse_server setting, allows for separate api/web client endpoints.

.gitignore

@@ -12,3 +12,5 @@ bin/
 coverage/
 vendor/
 /.gocache
+
+.vscode/

internal/cmd/epic/add/add.go

@@ -96,7 +96,7 @@ func add(cmd *cobra.Command, args []string) {
 		return nil
 	}()
 
-	msg := fmt.Sprintf("Issues added to the epic %s\n%s/browse/%s", params.epicKey, server, params.epicKey)
+	msg := fmt.Sprintf("Issues added to the epic %s\n%s", params.epicKey, cmdutil.GenerateServerBrowseURL(server, params.epicKey))
 
 	if projectType != jira.ProjectTypeNextGen {
 		cmdutil.ExitIfError(err)

internal/cmd/epic/create/create.go

@@ -128,7 +128,7 @@ func create(cmd *cobra.Command, _ []string) {
 	}()
 
 	cmdutil.ExitIfError(err)
-	cmdutil.Success("Epic created\n%s/browse/%s", server, key)
+	cmdutil.Success("Epic created\n%s", cmdutil.GenerateServerBrowseURL(server, key))
 
 	if web, _ := cmd.Flags().GetBool("web"); web {
 		err := cmdutil.Navigate(server, key)

internal/cmd/issue/assign/assign.go

@@ -115,7 +115,7 @@ func assign(cmd *cobra.Command, args []string) {
 	} else {
 		cmdutil.Success("User %q assigned to issue %q", uname, ac.params.key)
 	}
-	fmt.Printf("%s/browse/%s\n", viper.GetString("server"), ac.params.key)
+	fmt.Printf("%s\n", cmdutil.GenerateServerBrowseURL(viper.GetString("server"), ac.params.key))
 }
 
 type assignParams struct {

internal/cmd/issue/clone/clone.go

@@ -96,7 +96,7 @@ func clone(cmd *cobra.Command, args []string) {
 	}()
 	cmdutil.ExitIfError(err)
 
-	cmdutil.Success("Issue cloned\n%s/browse/%s", server, clonedIssueKey)
+	cmdutil.Success("Issue cloned\n%s", cmdutil.GenerateServerBrowseURL(server, clonedIssueKey))
 
 	var wg sync.WaitGroup
 	wg.Add(1)

internal/cmd/issue/comment/add/add.go

@@ -109,7 +109,7 @@ func add(cmd *cobra.Command, args []string) {
 	server := viper.GetString("server")
 
 	cmdutil.Success("Comment added to issue %q", ac.params.issueKey)
-	fmt.Printf("%s/browse/%s\n", server, ac.params.issueKey)
+	fmt.Printf("%s\n", cmdutil.GenerateServerBrowseURL(server, ac.params.issueKey))
 
 	if web, _ := cmd.Flags().GetBool("web"); web {
 		err := cmdutil.Navigate(server, ac.params.issueKey)

internal/cmd/issue/create/create.go

@@ -131,7 +131,7 @@ func create(cmd *cobra.Command, _ []string) {
 	}()
 
 	cmdutil.ExitIfError(err)
-	cmdutil.Success("Issue created\n%s/browse/%s", server, key)
+	cmdutil.Success("Issue created\n%s", cmdutil.GenerateServerBrowseURL(server, key))
 
 	if web, _ := cmd.Flags().GetBool("web"); web {
 		err := cmdutil.Navigate(server, key)

internal/cmd/issue/edit/edit.go

@@ -149,7 +149,7 @@ func edit(cmd *cobra.Command, args []string) {
 	}()
 	cmdutil.ExitIfError(err)
 
-	cmdutil.Success("Issue updated\n%s/browse/%s", server, params.issueKey)
+	cmdutil.Success("Issue updated\n%s", cmdutil.GenerateServerBrowseURL(server, params.issueKey))
 
 	handleUserAssign(project, params.issueKey, params.assignee, client)
 

internal/cmd/issue/link/link.go

@@ -82,7 +82,7 @@ func link(cmd *cobra.Command, args []string) {
 	server := viper.GetString("server")
 
 	cmdutil.Success("Issues linked as %q", lc.params.linkType)
-	fmt.Printf("%s/browse/%s\n", server, lc.params.inwardIssueKey)
+	fmt.Printf("%s\n", cmdutil.GenerateServerBrowseURL(server, lc.params.inwardIssueKey))
 
 	if web, _ := cmd.Flags().GetBool("web"); web {
 		err := cmdutil.Navigate(server, lc.params.inwardIssueKey)

internal/cmd/issue/link/remote/remote.go

@@ -61,7 +61,7 @@ func remotelink(cmd *cobra.Command, args []string) {
 	server := viper.GetString("server")
 
 	cmdutil.Success("Remote web link created for Issue %s", lc.params.issueKey)
-	fmt.Printf("%s/browse/%s\n", server, lc.params.issueKey)
+	fmt.Printf("%s\n", cmdutil.GenerateServerBrowseURL(server, lc.params.issueKey))
 
 	if web, _ := cmd.Flags().GetBool("web"); web {
 		err := cmdutil.Navigate(server, lc.params.issueKey)

internal/cmd/issue/move/move.go

@@ -119,7 +119,7 @@ func move(cmd *cobra.Command, args []string) {
 	server := viper.GetString("server")
 
 	cmdutil.Success("Issue transitioned to state %q", tr.Name)
-	fmt.Printf("%s/browse/%s\n", server, mc.params.key)
+	fmt.Printf("%s\n", cmdutil.GenerateServerBrowseURL(server, mc.params.key))
 
 	if web, _ := cmd.Flags().GetBool("web"); web {
 		err := cmdutil.Navigate(server, mc.params.key)

internal/cmd/issue/unlink/unlink.go

@@ -66,7 +66,7 @@ func unlink(cmd *cobra.Command, args []string) {
 	server := viper.GetString("server")
 
 	cmdutil.Success("Issues unlinked")
-	fmt.Printf("%s/browse/%s\n", server, uc.params.inwardIssueKey)
+	fmt.Printf("%s\n", cmdutil.GenerateServerBrowseURL(server, uc.params.inwardIssueKey))
 
 	if web, _ := cmd.Flags().GetBool("web"); web {
 		err := cmdutil.Navigate(server, uc.params.inwardIssueKey)

internal/cmd/issue/watch/watch.go

@@ -81,7 +81,7 @@ func watch(cmd *cobra.Command, args []string) {
 	cmdutil.ExitIfError(err)
 
 	cmdutil.Success("User %q added as watcher of issue %q", uname, ac.params.key)
-	fmt.Printf("%s/browse/%s\n", viper.GetString("server"), ac.params.key)
+	fmt.Printf("%s\n", cmdutil.GenerateServerBrowseURL(viper.GetString("server"), ac.params.key))
 }
 
 type watchParams struct {

internal/cmd/issue/worklog/add/add.go

@@ -104,7 +104,7 @@ func add(cmd *cobra.Command, args []string) {
 	server := viper.GetString("server")
 
 	cmdutil.Success("Worklog added to issue %q", ac.params.issueKey)
-	fmt.Printf("%s/browse/%s\n", server, ac.params.issueKey)
+	fmt.Printf("%s\n", cmdutil.GenerateServerBrowseURL(server, ac.params.issueKey))
 }
 
 type addParams struct {

internal/cmd/open/open.go

@@ -43,9 +43,9 @@ func open(cmd *cobra.Command, args []string) {
 	var url string
 
 	if len(args) == 0 {
-		url = fmt.Sprintf("%s/browse/%s", server, project)
+		url = cmdutil.GenerateServerBrowseURL(server, project)
 	} else {
-		url = fmt.Sprintf("%s/browse/%s", server, cmdutil.GetJiraIssueKey(project, args[0]))
+		url = cmdutil.GenerateServerBrowseURL(server, cmdutil.GetJiraIssueKey(project, args[0]))
 	}
 
 	fmt.Println(url)

internal/cmd/sprint/add/add.go

@@ -70,7 +70,7 @@ func add(cmd *cobra.Command, args []string) {
 	}()
 	cmdutil.ExitIfError(err)
 
-	cmdutil.Success(fmt.Sprintf("Issues added to the sprint %s\n%s/browse/%s", params.sprintID, server, project))
+	cmdutil.Success(fmt.Sprintf("Issues added to the sprint %s\n%s", params.sprintID, cmdutil.GenerateServerBrowseURL(server, project)))
 }
 
 func parseFlags(flags query.FlagParser, args []string, project string) *addParams {

internal/cmdutil/utils.go

@@ -91,10 +91,20 @@ func Failed(msg string, args ...interface{}) {
 
 // Navigate navigates to jira issue.
 func Navigate(server, path string) error {
-	url := fmt.Sprintf("%s/browse/%s", server, path)
+	url := GenerateServerBrowseURL(server, path)
 	return browser.Browse(url)
 }
 
+// GenerateServerBrowseURL will return the `browse` URL for a given key.
+// The server section can be overridden via `browse_server` in config.
+// This is useful if your API endpoint is separate from the web client endpoint.
+func GenerateServerBrowseURL(server, key string) string {
+	if viper.GetString("browse_server") != "" {
+		server = viper.GetString("browse_server")
+	}
+	return fmt.Sprintf("%s/browse/%s", server, key)
+}
+
 // FormatDateTimeHuman formats date time in human readable format.
 func FormatDateTimeHuman(dt, format string) string {
 	t, err := time.Parse(format, dt)

internal/view/helper.go

@@ -13,6 +13,7 @@ import (
 	"github.com/fatih/color"
 	"github.com/mgutz/ansi"
 
+	"github.com/ankitpokhrel/jira-cli/internal/cmdutil"
 	"github.com/ankitpokhrel/jira-cli/pkg/browser"
 	"github.com/ankitpokhrel/jira-cli/pkg/tui"
 )
@@ -129,7 +130,7 @@ func issueKeyFromTuiData(r int, d interface{}) string {
 }
 
 func jiraURLFromTuiData(server string, r int, d interface{}) string {
-	return fmt.Sprintf("%s/browse/%s", server, issueKeyFromTuiData(r, d))
+	return cmdutil.GenerateServerBrowseURL(server, issueKeyFromTuiData(r, d))
 }
 
 func navigate(server string) tui.SelectedFunc {

internal/view/issue.go

@@ -430,7 +430,7 @@ func (i Issue) footer() string {
 	if i.Display.Plain {
 		out.WriteString("\n")
 	}
-	out.WriteString(gray(fmt.Sprintf("View this issue on Jira: %s/browse/%s", i.Server, i.Data.Key)))
+	out.WriteString(gray(fmt.Sprintf("View this issue on Jira: %s", cmdutil.GenerateServerBrowseURL(i.Server, i.Data.Key))))
 
 	return out.String()
 }