chore: Replace fetchData with http.FetchJSON

Author: mirkobrombin

docs/articles/guide/api-integration.md

@@ -4,21 +4,31 @@
 Applications often need to communicate with external services. The [HTTP API](../api/http) provides helpers that cache responses and surface loading states.
 
 ```go
-var todo Todo
+var todo struct {
+    Title string `json:"title"`
+}
 if err := http.FetchJSON("https://jsonplaceholder.typicode.com/todos/1", &todo); err != nil {
     if errors.Is(err, http.ErrPending) {
         // display loading UI
+    } else {
+        // handle fetch error
     }
 }
 ```
 
 ## When to Use
-Use API calls when retrieving remote data to populate stores or render components.
+Use `http.FetchJSON` when retrieving remote data to populate stores or render components.
 
 ```go
 go func() {
-    data, _ := fetchData("https://jsonplaceholder.typicode.com/todos/1")
-    store.Set("apiData", data)
+    var todo struct {
+        Title string `json:"title"`
+    }
+    if err := http.FetchJSON("https://jsonplaceholder.typicode.com/todos/1", &todo); err != nil {
+        store.Set("apiData", err.Error())
+        return
+    }
+    store.Set("apiData", todo.Title)
 }()
 ```
 

docs/examples/api_integration_test.go

@@ -7,20 +7,23 @@ import (
 	"net/http/httptest"
 	"testing"
 
-	"github.com/rfwlab/rfw/docs/examples/components"
+	rfwhttp "github.com/rfwlab/rfw/v1/http"
 )
 
-func TestFetchData(t *testing.T) {
+func TestFetchJSON(t *testing.T) {
 	srv := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
-		w.Write([]byte("ok"))
+		w.Header().Set("Content-Type", "application/json")
+		w.Write([]byte(`{"title":"ok"}`))
 	}))
 	defer srv.Close()
 
-	data, err := components.FetchData(srv.URL)
-	if err != nil {
+	var todo struct {
+		Title string `json:"title"`
+	}
+	if err := rfwhttp.FetchJSON(srv.URL, &todo); err != nil {
 		t.Fatalf("unexpected error: %v", err)
 	}
-	if data != "ok" {
-		t.Fatalf("expected 'ok', got %s", data)
+	if todo.Title != "ok" {
+		t.Fatalf("expected 'ok', got %s", todo.Title)
 	}
 }

docs/examples/components/api_integration_component.go

@@ -4,11 +4,10 @@ package components
 
 import (
 	_ "embed"
-	"io"
-	"net/http"
 
 	core "github.com/rfwlab/rfw/v1/core"
 	dom "github.com/rfwlab/rfw/v1/dom"
+	"github.com/rfwlab/rfw/v1/http"
 )
 
 //go:embed templates/api_integration_component.rtml
@@ -18,27 +17,16 @@ func NewAPIIntegrationComponent() *core.HTMLComponent {
 	c := core.NewComponent("APIIntegrationComponent", apiIntegrationComponentTpl, nil)
 	dom.RegisterHandlerFunc("load", func() {
 		go func() {
-			data, err := FetchData("https://jsonplaceholder.typicode.com/todos/1")
-			if err != nil {
+			var todo struct {
+				Title string `json:"title"`
+			}
+			if err := http.FetchJSON("https://jsonplaceholder.typicode.com/todos/1", &todo); err != nil {
 				c.Store.Set("apiData", err.Error())
 				return
 			}
-			c.Store.Set("apiData", data)
+			c.Store.Set("apiData", todo.Title)
 		}()
 	})
 
 	return c
 }
-
-func FetchData(url string) (string, error) {
-	resp, err := http.Get(url)
-	if err != nil {
-		return "", err
-	}
-	defer resp.Body.Close()
-	b, err := io.ReadAll(resp.Body)
-	if err != nil {
-		return "", err
-	}
-	return string(b), nil
-}