Added better error handling on time entries

mightymatth · Jan 14, 2020 · 7645df1 · 7645df1
1 parent b30f47d
commit 7645df1
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 3 deletions.
diff --git a/client/timeentry.go b/client/timeentry.go
@@ -35,12 +35,17 @@ func (c *Client) GetTimeEntries(queryParams string) ([]TimeEntry, error) {
 	}
 
 	var response TimeEntriesResponse
-	_, err = c.Do(req, &response)
+	res, err := c.Do(req, &response)
 	if err != nil {
 		return nil, err
 	}
 
-	return response.TimeEntries, nil
+	switch res.StatusCode {
+	case http.StatusOK:
+		return response.TimeEntries, nil
+	default:
+		return nil, fmt.Errorf("cannot get time entries (status %v)", res.StatusCode)
+	}
 }
 
 type TimeEntryBody struct {

diff --git a/cmd/status.go b/cmd/status.go
@@ -108,7 +108,7 @@ func statusFunc(_ *cobra.Command, _ []string) {
 	fmt.Printf("[%d] %s %s (%s)\n", u.Id, u.FirstName, u.LastName, u.Email)
 
 	t := utils.NewTable()
-	t.AppendRow(table.Row{"Period", "Hours"})
+	t.AppendHeader(table.Row{"Period", "Hours"})
 	t.AppendRow(table.Row{"Today", <-today})
 	t.AppendRow(table.Row{"Yesterday", <-yesterday})
 	t.AppendRow(table.Row{"This week", <-thisWeek})