[GH-61]: Add tooltip for pull request and issue links

Makefile

@@ -88,7 +88,7 @@ endif
 ## Ensures NPM dependencies are installed without having to run this all the time.
 webapp/node_modules: $(wildcard webapp/package.json)
 ifneq ($(HAS_WEBAPP),)
-	cd webapp && $(NPM) install
+	cd webapp && $(NPM) install --verbose
 	touch $@
 endif
 

server/api.go

@@ -55,6 +55,8 @@ func (p *Plugin) initializeAPI() {
 	apiRouter.HandleFunc("/yourprs", p.checkAuth(p.attachUserContext(p.getYourPrs), ResponseTypePlain)).Methods(http.MethodGet)
 	apiRouter.HandleFunc("/yourassignments", p.checkAuth(p.attachUserContext(p.getYourAssignments), ResponseTypePlain)).Methods(http.MethodGet)
 	apiRouter.HandleFunc("/unreads", p.checkAuth(p.attachUserContext(p.getUnreads), ResponseTypePlain)).Methods(http.MethodGet)
+	apiRouter.HandleFunc("/issue", p.checkAuth(p.attachUserContext(p.getIssueByNumber), ResponseTypeJSON)).Methods(http.MethodGet)
+	apiRouter.HandleFunc("/mergerequest", p.checkAuth(p.attachUserContext(p.getMergeRequestByNumber), ResponseTypeJSON)).Methods(http.MethodGet)
 
 	apiRouter.HandleFunc("/settings", p.checkAuth(p.attachUserContext(p.updateSettings), ResponseTypePlain)).Methods(http.MethodPost)
 }
@@ -595,3 +597,43 @@ func (p *Plugin) updateSettings(c *UserContext, w http.ResponseWriter, r *http.R
 
 	p.writeAPIResponse(w, info.Settings)
 }
+
+func (p *Plugin) getIssueByNumber(c *UserContext, w http.ResponseWriter, r *http.Request) {
+	owner := r.FormValue("owner")
+	repo := r.FormValue("repo")
+	issueID, err := strconv.Atoi(r.FormValue("number"))
+	if err != nil {
+		c.Log.WithError(err).Warnf("Unable to convert issueID into int")
+		p.writeAPIError(w, &APIErrorResponse{ID: "", Message: "Unable to convert issueID into int.", StatusCode: http.StatusInternalServerError})
+		return
+	}
+
+	issue, err := p.GitlabClient.GetIssueByID(c.Ctx, c.GitlabInfo, owner, repo, issueID)
+	if err != nil {
+		c.Log.WithError(err).Warnf("Unable to get issue in GitLab API")
+		p.writeAPIError(w, &APIErrorResponse{ID: "", Message: "Unable to get issue in GitLab API.", StatusCode: http.StatusInternalServerError})
+		return
+	}
+
+	p.writeAPIResponse(w, issue)
+}
+
+func (p *Plugin) getMergeRequestByNumber(c *UserContext, w http.ResponseWriter, r *http.Request) {
+	owner := r.FormValue("owner")
+	repo := r.FormValue("repo")
+	mergeRequestID, err := strconv.Atoi(r.FormValue("number"))
+	if err != nil {
+		c.Log.WithError(err).Warnf("Unable to convert mergeRequestID into int")
+		p.writeAPIError(w, &APIErrorResponse{ID: "", Message: "Unable to convert mergeRequestID into int.", StatusCode: http.StatusInternalServerError})
+		return
+	}
+
+	mergeRequest, err := p.GitlabClient.GetMergeRequestByID(c.Ctx, c.GitlabInfo, owner, repo, mergeRequestID)
+	if err != nil {
+		c.Log.WithError(err).Warnf("Unable to get merge request in GitLab API")
+		p.writeAPIError(w, &APIErrorResponse{ID: "", Message: "Unable to get merge request in GitLab API.", StatusCode: http.StatusInternalServerError})
+		return
+	}
+
+	p.writeAPIResponse(w, mergeRequest)
+}

server/gitlab/api.go

@@ -16,6 +16,16 @@ const (
 	scopeAll    = "all"
 )
 
+type Issue struct {
+	*internGitlab.Issue
+	LabelsWithDetails []*internGitlab.Label `json:"labels_with_details,omitempty"`
+}
+
+type MergeRequest struct {
+	*internGitlab.MergeRequest
+	LabelsWithDetails []*internGitlab.Label `json:"labels_with_details,omitempty"`
+}
+
 // NewGroupHook creates a webhook associated with a GitLab group
 func (g *gitlab) NewGroupHook(ctx context.Context, user *UserInfo, groupName string, webhookOptions *AddWebhookOptions) (*WebhookInfo, error) {
 	client, err := g.gitlabConnect(*user.Token)
@@ -473,3 +483,75 @@ func (g *gitlab) ResolveNamespaceAndProject(
 	}
 	return "", "", ErrNotFound
 }
+
+func (g *gitlab) GetIssueByID(ctx context.Context, user *UserInfo, owner, repo string, issueID int) (*Issue, error) {
+	client, err := g.gitlabConnect(*user.Token)
+	if err != nil {
+		return nil, err
+	}
+	projectPath := fmt.Sprintf("%s/%s", owner, repo)
+	issue, resp, err := client.Issues.GetIssue(projectPath, issueID)
+	if respErr := checkResponse(resp); respErr != nil {
+		return nil, respErr
+	}
+	if err != nil {
+		return nil, errors.Wrap(err, "can't get issue in GitLab api")
+	}
+
+	gitlabIssue := &Issue{
+		Issue: issue,
+	}
+	if issue.Labels != nil {
+		labelsWithDetails, err := g.GetLabelDetails(client, projectPath, issue.Labels)
+		if err != nil {
+			return nil, err
+		}
+		gitlabIssue.LabelsWithDetails = labelsWithDetails
+	}
+
+	return gitlabIssue, nil
+}
+
+func (g *gitlab) GetMergeRequestByID(ctx context.Context, user *UserInfo, owner, repo string, mergeRequestID int) (*MergeRequest, error) {
+	client, err := g.gitlabConnect(*user.Token)
+	if err != nil {
+		return nil, err
+	}
+	projectPath := fmt.Sprintf("%s/%s", owner, repo)
+	mergeRequest, resp, err := client.MergeRequests.GetMergeRequest(projectPath, mergeRequestID, nil)
+	if respErr := checkResponse(resp); respErr != nil {
+		return nil, respErr
+	}
+	if err != nil {
+		return nil, errors.Wrap(err, "can't get merge request in GitLab api")
+	}
+
+	gitlabMergeRequest := &MergeRequest{
+		MergeRequest: mergeRequest,
+	}
+	if mergeRequest.Labels != nil {
+		labelsWithDetails, err := g.GetLabelDetails(client, projectPath, mergeRequest.Labels)
+		if err != nil {
+			return nil, err
+		}
+		gitlabMergeRequest.LabelsWithDetails = labelsWithDetails
+	}
+
+	return gitlabMergeRequest, nil
+}
+
+func (g *gitlab) GetLabelDetails(client *internGitlab.Client, projectPath string, labels internGitlab.Labels) ([]*internGitlab.Label, error) {
+	var labelsWithDetails []*internGitlab.Label
+	for _, label := range labels {
+		labelWithDetails, resp, err := client.Labels.GetLabel(projectPath, label)
+		if respErr := checkResponse(resp); respErr != nil {
+			return nil, respErr
+		}
+		if err != nil {
+			return nil, errors.Wrap(err, "can't get label in GitLab api")
+		}
+		labelsWithDetails = append(labelsWithDetails, labelWithDetails)
+	}
+
+	return labelsWithDetails, nil
+}

server/gitlab/gitlab.go

@@ -32,6 +32,8 @@ type Gitlab interface {
 	GetGroupHooks(ctx context.Context, user *UserInfo, owner string) ([]*WebhookInfo, error)
 	NewProjectHook(ctx context.Context, user *UserInfo, projectID interface{}, projectHookOptions *AddWebhookOptions) (*WebhookInfo, error)
 	NewGroupHook(ctx context.Context, user *UserInfo, groupName string, groupHookOptions *AddWebhookOptions) (*WebhookInfo, error)
+	GetIssueByID(ctx context.Context, user *UserInfo, owner, repo string, issueID int) (*Issue, error)
+	GetMergeRequestByID(ctx context.Context, user *UserInfo, owner, repo string, mergeRequestID int) (*MergeRequest, error)
 	// ResolveNamespaceAndProject accepts full path to User, Group or namespaced Project and returns corresponding
 	// namespace and project name.
 	//