Merge pull request #95 from thestormforge/fix/activity-patch-request

Add logic for passing a correct activity patch request body

Author: jgustie

pkg/api/applications/v2/activity.go

@@ -105,6 +105,12 @@ type RefreshActivity struct {
 	ActivityFailure
 }
 
+type ActivityPatchRequest struct {
+	Title string `json:"title"`
+	// Data is a JSON-serializable value for internal metadata about the Activity
+	Data any `json:"_stormforge,omitempty"`
+}
+
 type ActivityFailure struct {
 	FailureReason  string `json:"failure_reason,omitempty"`
 	FailureMessage string `json:"failure_message,omitempty"`

pkg/api/applications/v2/api.go

@@ -108,7 +108,7 @@ type API interface {
 	// DeleteActivity resolves application activity.
 	DeleteActivity(ctx context.Context, u string) error
 	// PatchApplicationActivity updates application activity.
-	PatchApplicationActivity(ctx context.Context, u string, a ActivityFailure) error
+	PatchApplicationActivity(ctx context.Context, u string, a ActivityPatchRequest) error
 
 	// SubscribeActivity returns a subscriber for the activity feed.
 	SubscribeActivity(ctx context.Context, q ActivityFeedQuery) (Subscriber, error)

pkg/api/applications/v2/http.go

@@ -592,7 +592,7 @@ func (h *httpAPI) DeleteActivity(ctx context.Context, u string) error {
 	}
 }
 
-func (h *httpAPI) PatchApplicationActivity(ctx context.Context, u string, a ActivityFailure) error {
+func (h *httpAPI) PatchApplicationActivity(ctx context.Context, u string, a ActivityPatchRequest) error {
 	req, err := httpNewJSONRequest(http.MethodPatch, u, a)
 	if err != nil {
 		return err