Add the ability to start the tracker when submitting an entry

.gitignore

@@ -1,2 +1,2 @@
-# Added by goreleaser init:
 dist/
+.DS_Store

mite/mite.go

@@ -51,8 +51,10 @@ type Project struct {
 func (p Project) String() string {
 	return fmt.Sprintf("%d: %s (%s)", p.ID, p.Name, p.CustomerName)
 }
-func (p Project) GetID() int      { return p.ID }
+func (p Project) GetID() int { return p.ID }
-func (p Project) GetName() string { return p.Name + " (" + p.CustomerName + ")" }
+func (p Project) GetName() string {
+	return p.Name + " (" + p.CustomerName + ")"
+}
 
 func (a APIClient) GetProjects() (Projects, error) {
 	// GET /projects.json
@@ -238,7 +240,9 @@ type requestAddTimeEntry struct {
 	} `json:"time_entry"`
 }
 
-func (a APIClient) AddTimeEntry(date string, minutes int, notes string, projectId, serviceId int) error {
+type apiPostTimeEntry timeEntryHolder
+
+func (a APIClient) AddTimeEntry(date string, minutes int, notes string, projectId, serviceId int) (int, error) {
 	// POST /time_entries.json
 	// {
 	// 	"time_entry": {
@@ -254,8 +258,10 @@ func (a APIClient) AddTimeEntry(date string, minutes int, notes string, projectI
 	req.RequestTimeEntryHolder.ProjectID = projectId
 	req.RequestTimeEntryHolder.ServiceID = serviceId
 
-	err := a.post("/time_entries.json", req)
+	res := apiPostTimeEntry{}
-	return err
+
+	err := a.post("/time_entries.json", req, &res)
+	return res.TimeEntry.ID, err
 }
 
 type apiTimeTrackerEntry struct {
@@ -264,6 +270,7 @@ type apiTimeTrackerEntry struct {
 
 type timeTrackerHolder struct {
 	TrackingTimeEntry *TrackingTimeEntry `json:"tracking_time_entry"`
+	StoppedTimeEntry  *StoppedTimeEntry  `json:"stopped_time_entry"`
 }
 
 type TrackingTimeEntry struct {
@@ -272,6 +279,25 @@ type TrackingTimeEntry struct {
 	Since   time.Time `json:"since"`
 }
 
+type StoppedTimeEntry struct {
+	ID      int `json:"id"`
+	Minutes int `json:"minutes"`
+}
+
+// {
+// 	"tracker": {
+// 	  "tracking_time_entry": {
+// 		"id": 36135322,
+// 		"minutes": 0,
+// 		"since": "2015-10-15T17:33:52+02:00"
+// 	  },
+// 	 "stopped_time_entry": {
+// 		"id": 36134329,
+// 		"minutes": 46
+// 	  }
+// 	}
+//   }
+
 // {
 // 	"tracker": {
 // 	  "tracking_time_entry": {
@@ -298,9 +324,89 @@ func (a APIClient) GetTimeTracker() (TrackingTimeEntry, error) {
 	return *r.TimeTrackerHolder.TrackingTimeEntry, nil
 }
 
-func (a APIClient) post(path string, data any) error {
+func (a APIClient) StartTimeTracker(id int) (TrackingTimeEntry, *StoppedTimeEntry, error) {
+	url := fmt.Sprintf("/tracker/%d.json", id)
+	r := apiTimeTrackerEntry{}
 
-	b, err := json.Marshal(data)
+	err := a.patch(url, &r)
+
+	if err != nil {
+		return TrackingTimeEntry{}, nil, err
+	}
+	if r.TimeTrackerHolder.TrackingTimeEntry == nil {
+		// I don't think this should happen, the patch should have been a 404?
+
+		panic(fmt.Sprintf("unexpected failure to find a tracking entry in a successful PATCH\n%#v", r.TimeTrackerHolder))
+	}
+	return *r.TimeTrackerHolder.TrackingTimeEntry, r.TimeTrackerHolder.StoppedTimeEntry, nil
+}
+
+func (a APIClient) StopTimeTracker(id int) (StoppedTimeEntry, error) {
+	url := fmt.Sprintf("/tracker/%d.json", id)
+	r := apiTimeTrackerEntry{}
+
+	err := a.delete(url, &r)
+
+	if err != nil {
+		return StoppedTimeEntry{}, err
+	}
+	if r.TimeTrackerHolder.StoppedTimeEntry == nil {
+		// I don't think this should happen, the patch should have been a 404?
+
+		panic(fmt.Sprintf("unexpected failure to find a tracking entry in a successful DELETE\n%#v", r.TimeTrackerHolder))
+	}
+	return *r.TimeTrackerHolder.StoppedTimeEntry, nil
+}
+
+func (a APIClient) delete(path string, data any) error {
+
+	req, err := http.NewRequest("DELETE", baseurl(a.domain, path), nil)
+	if err != nil {
+		return err
+	}
+	req.Header.Add("X-MiteApiKey", a.apiKey)
+	resp, err := http.DefaultClient.Do(req)
+
+	if err != nil {
+		return err
+	}
+	defer resp.Body.Close()
+	if resp.StatusCode >= 300 {
+		return fmt.Errorf("expected 2XX, got %d", resp.StatusCode)
+	}
+	err = json.NewDecoder(resp.Body).Decode(&data)
+	if err != nil {
+		return err
+	}
+	return nil
+}
+
+func (a APIClient) patch(path string, data any) error {
+
+	req, err := http.NewRequest("PATCH", baseurl(a.domain, path), nil)
+	if err != nil {
+		return err
+	}
+	req.Header.Add("X-MiteApiKey", a.apiKey)
+	resp, err := http.DefaultClient.Do(req)
+
+	if err != nil {
+		return err
+	}
+	defer resp.Body.Close()
+	if resp.StatusCode >= 300 {
+		return fmt.Errorf("expected 2XX, got %d", resp.StatusCode)
+	}
+	err = json.NewDecoder(resp.Body).Decode(&data)
+	if err != nil {
+		return err
+	}
+	return nil
+}
+
+func (a APIClient) post(path string, dataIn any, dataOut any) error {
+
+	b, err := json.Marshal(dataIn)
 	if err != nil {
 		return err
 	}
@@ -321,6 +427,11 @@ func (a APIClient) post(path string, data any) error {
 		return fmt.Errorf("expected 2XX, got %d", resp.StatusCode)
 	}
 
+	err = json.NewDecoder(resp.Body).Decode(&dataOut)
+	if err != nil {
+		return err
+	}
+
 	return nil
 
 }

mite/mite_test.go

@@ -6,19 +6,10 @@ import (
 )
 
 func TestUnmarshalTimeTracking(t *testing.T) {
-	trackerOn := []byte(`
+
-{
-	"tracker": {
-	  "tracking_time_entry": {
-		"id": 36135321,
-		"minutes": 247,
-		"since": "2015-10-15T17:05:04+02:00"
-	  }
-	}
-  }`)
 	trackerOff := []byte(`{
-  "tracker": {}
+	"tracker": {}
-}`)
+  }`)
 	off := apiTimeTrackerEntry{}
 	err := json.Unmarshal(trackerOff, &off)
 	if err != nil {
@@ -28,14 +19,23 @@ func TestUnmarshalTimeTracking(t *testing.T) {
 			t.Error("expected nil, but is not")
 		}
 	}
-
+	trackerOn := []byte(`
+	{
+		"tracker": {
+		  "tracking_time_entry": {
+			"id": 36135321,
+			"minutes": 247,
+			"since": "2015-10-15T17:05:04+02:00"
+		  }
+		}
+	  }`)
 	on := apiTimeTrackerEntry{}
 	err = json.Unmarshal(trackerOn, &on)
 	if err != nil {
 		t.Error(err)
 	} else {
 		if on.TimeTrackerHolder.TrackingTimeEntry == nil {
-			t.Error("expected note nil, but is not")
+			t.Error("expected not nil, but is not")
 		} else {
 			if on.TimeTrackerHolder.TrackingTimeEntry.ID != 36135321 {
 				t.Error("bad unmarshal?")
@@ -43,4 +43,40 @@ func TestUnmarshalTimeTracking(t *testing.T) {
 		}
 	}
 
+	trackerStart := []byte(`{
+  "tracker": {
+    "tracking_time_entry": {
+      "id": 36135322,
+      "minutes": 0,
+      "since": "2015-10-15T17:33:52+02:00"
+    },
+   "stopped_time_entry": {
+      "id": 36134329,
+      "minutes": 46
+    }
+  }
+}`)
+	start := apiTimeTrackerEntry{}
+	err = json.Unmarshal(trackerStart, &start)
+	if err != nil {
+		t.Error(err)
+	} else {
+		if start.TimeTrackerHolder.TrackingTimeEntry == nil {
+			t.Error("expected not nil, but is not")
+		} else {
+			if start.TimeTrackerHolder.TrackingTimeEntry.ID != 36135322 {
+				t.Error("bad unmarshal?")
+			}
+		}
+
+		if start.TimeTrackerHolder.StoppedTimeEntry == nil {
+			t.Error("expected not nil, but is nil")
+		} else {
+			if start.TimeTrackerHolder.StoppedTimeEntry.ID != 36134329 {
+				t.Error("bad unmarshal")
+			}
+		}
+
+	}
+
 }

model_form.go

@@ -35,6 +35,7 @@ func (m model) updateForm(msg tea.Msg) (tea.Model, tea.Cmd) {
 		projectID := m.formData.form.GetString("project")
 		description := m.formData.form.GetString("description")
 		minutes := m.formData.form.GetString("minutes")
+		tracker := m.formData.form.GetBool("tracker")
 
 		minutesInt, err1 := strconv.ParseInt(minutes, 10, 64)
 		serviceIDInt, err2 := strconv.ParseInt(serviceID, 10, 64)
@@ -45,14 +46,27 @@ func (m model) updateForm(msg tea.Msg) (tea.Model, tea.Cmd) {
 			m.tuiMode = MODE_CAL
 		} else {
 
-			err := m.miteAPI.AddTimeEntry(m.dest.Format(time.DateOnly), int(minutesInt), description, int(projectIDInt), int(serviceIDInt))
+			id, err := m.miteAPI.AddTimeEntry(m.dest.Format(time.DateOnly), int(minutesInt), description, int(projectIDInt), int(serviceIDInt))
 			if err != nil {
 				m.statusBarMessage = err.Error()
 				m.tuiMode = MODE_CAL
 			} else {
-				m.statusBarMessage = "Successfully logged time"
+				if tracker {
-				m.tuiMode = MODE_CAL
+					_, _, err = m.miteAPI.StartTimeTracker(id)
-				return m, m.fetchMiteData()
+					if err != nil {
+						m.statusBarMessage = err.Error()
+						m.tuiMode = MODE_CAL
+					} else {
+						m.statusBarMessage = "Successfully logged time and started tracker"
+						m.tuiMode = MODE_CAL
+						return m, m.fetchMiteData()
+					}
+				} else {
+
+					m.statusBarMessage = "Successfully logged time"
+					m.tuiMode = MODE_CAL
+					return m, m.fetchMiteData()
+				}
 			}
 		}
 
@@ -101,38 +115,51 @@ func (m model) buildForm() *huh.Form {
 		Key("project").
 		Title("Project").
 		Options(projOptions...).
-		Height(10)
+		Height(6)
+
+	description := huh.NewText().
+		Key("description").
+		Title("description").
+		Validate(func(s string) error {
+			if s == "" {
+				return errors.New("must enter a description")
+			}
+			return nil
+		})
+
+	defMinutes := "0"
+	minutes := huh.NewInput().
+		Key("minutes").
+		CharLimit(5).
+		Validate(
+			func(s string) error {
+				h, err := strconv.ParseInt(s, 10, 64)
+				if err != nil {
+					return err
+				}
+				if h < 0 {
+					return errors.New("must be positive")
+				}
+				return err
+			}).
+		Title("Minutes").
+		Value(&defMinutes)
+
+	confirmTracker := huh.NewConfirm().
+		Key("tracker").
+		Affirmative("Yes").
+		Negative("No").
+		Title("Start tracker")
 
 	form := huh.NewForm(
 		huh.NewGroup(
 			pl,
 			sl,
+			description,
 		),
 		huh.NewGroup(
-			huh.NewText().
+			minutes,
-				Key("description").
+			confirmTracker,
-				Title("description").
-				Validate(func(s string) error {
-					if s == "" {
-						return errors.New("must enter a description")
-					}
-					return nil
-				}),
-			huh.NewInput().
-				Key("minutes").
-				CharLimit(5).
-				Validate(
-					func(s string) error {
-						h, err := strconv.ParseInt(s, 10, 64)
-						if err != nil {
-							return err
-						}
-						if h < 0 {
-							return errors.New("must be positive")
-						}
-						return err
-					}).
-				Title("Minutes"),
 		),
 	)
 	return form