move loader closer to fetch

jzyinq · Mar 25, 2024 · b520a52 · b520a52
1 parent 5d14089
commit b520a52
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/gojira/cli.go b/gojira/cli.go
@@ -44,7 +44,9 @@ func NewWorkLogIssues() error {
 		return nil
 	}
 	if app.workLogsIssues.startDate != startDate || app.workLogsIssues.endDate != endDate {
+		app.ui.loaderView.Show("Fetching worklogs...")
 		app.workLogs, err = GetWorkLogs()
+		app.ui.loaderView.Hide()
 		if err != nil {
 			return err
 		}

diff --git a/gojira/dayview.go b/gojira/dayview.go
@@ -103,13 +103,11 @@ func loadWorklogs() {
 	case loadingWorklogs <- true:
 		go func() {
 			defer func() { <-loadingWorklogs }()
-			app.ui.loaderView.Show("Fetching worklogs...")
 			err := NewWorkLogIssues()
 			if err != nil {
 				app.ui.errorView.ShowError(err.Error())
 			}
 			app.ui.dayView.update()
-			app.ui.loaderView.Hide()
 		}()
 	default:
 		// The goroutine is already loadingWorklogs, do nothing