fix time panic

taraspos · taraspos · commit ad8257bd7d66 · 2024-12-11T13:04:09.000Z
diff --git a/tools/amplify-preview/amplify.go b/tools/amplify-preview/amplify.go
@@ -141,7 +141,7 @@ func (amp *AmplifyPreview) StartJob(ctx context.Context, branch *types.Branch) (
 		return nil, err
 	}
 
-	logger.Info("Successfully started job", logKeyAppID, appID, logKeyBranchName, branch.BranchName, logKeyJobID, resp.JobSummary.JobId)
+	logger.Info("Successfully started job", logKeyAppID, appID, logKeyBranchName, *branch.BranchName, logKeyJobID, *resp.JobSummary.JobId)
 
 	return resp.JobSummary, nil
 
@@ -216,6 +216,9 @@ func amplifyJobToMarkdown(job *types.JobSummary, branch *types.Branch) string {
 	if job.EndTime != nil {
 		updateTime = job.EndTime
 	}
+	if updateTime == nil {
+		updateTime = branch.CreateTime
+	}
 
 	commentBody.WriteString(amplifyMarkdownHeader)
 	commentBody.WriteByte('\n')

Original file line number	Diff line number	Diff line change
`@@ -141,7 +141,7 @@ func (amp AmplifyPreview) StartJob(ctx context.Context, branch types.Branch) (`
`141`	`141`	`return nil, err`
`142`	`142`	`}`
`143`	`143`
`144`		`- logger.Info("Successfully started job", logKeyAppID, appID, logKeyBranchName, branch.BranchName, logKeyJobID, resp.JobSummary.JobId)`
	`144`	`+ logger.Info("Successfully started job", logKeyAppID, appID, logKeyBranchName, branch.BranchName, logKeyJobID, resp.JobSummary.JobId)`
`145`	`145`
`146`	`146`	`return resp.JobSummary, nil`
`147`	`147`
`@@ -216,6 +216,9 @@ func amplifyJobToMarkdown(job types.JobSummary, branch types.Branch) string {`
`216`	`216`	`if job.EndTime != nil {`
`217`	`217`	`updateTime = job.EndTime`
`218`	`218`	`}`
	`219`	`+ if updateTime == nil {`
	`220`	`+ updateTime = branch.CreateTime`
	`221`	`+ }`
`219`	`222`
`220`	`223`	`commentBody.WriteString(amplifyMarkdownHeader)`
`221`	`224`	`commentBody.WriteByte('\n')`