From d87ed549e9ac21fa5fa38d521c028c1011956de7 Mon Sep 17 00:00:00 2001 From: Spencer Watson Date: Wed, 1 Jan 2025 11:16:46 -0800 Subject: [PATCH 1/2] Fix loading design by URL --- src/api/kitePaint/getDesign.js | 11 ++++++++--- 1 file changed, 8 insertions(+), 3 deletions(-) diff --git a/src/api/kitePaint/getDesign.js b/src/api/kitePaint/getDesign.js index e195792..c2d3853 100644 --- a/src/api/kitePaint/getDesign.js +++ b/src/api/kitePaint/getDesign.js @@ -1,3 +1,4 @@ +import Qs from "qs"; import Design from "../../models/Design"; import { error } from "../../theme/Alert"; @@ -15,11 +16,15 @@ export default async function getDesign(id, useCache = true) { return cache; } } - const response = await this.axiosInstance.get("/designs.php", { - params: { + const requestData = { + filter: { id } - }); + }; + const requestString = Qs.stringify(requestData); + const response = await this.axiosInstance.get( + `/designs.php?${requestString}` + ); // Handle invalid responses if (!response.data || !response.data.length) { From 13ab714934e2436a982c7201e3189a815664790a Mon Sep 17 00:00:00 2001 From: Spencer Watson Date: Wed, 1 Jan 2025 11:20:01 -0800 Subject: [PATCH 2/2] fix test --- src/api/kitePaint/__unit__/getDesign.test.js | 10 +++------- 1 file changed, 3 insertions(+), 7 deletions(-) diff --git a/src/api/kitePaint/__unit__/getDesign.test.js b/src/api/kitePaint/__unit__/getDesign.test.js index a7c78ef..777944d 100644 --- a/src/api/kitePaint/__unit__/getDesign.test.js +++ b/src/api/kitePaint/__unit__/getDesign.test.js @@ -11,14 +11,10 @@ describe("#getDesign", () => { }; }); it("makes the relevant request", async () => { - expect.assertions(2); await Api.getDesign("abc").catch(() => {}); - expect(Api.axiosInstance.get.mock.calls[0][0]).toEqual("/designs.php"); - expect(Api.axiosInstance.get.mock.calls[0][1]).toEqual({ - params: { - id: "abc" - } - }); + expect(Api.axiosInstance.get.mock.calls[0][0]).toEqual( + "/designs.php?filter%5Bid%5D=abc" + ); }); it("does not make identical requests when they have been cached", async () => { expect.assertions(1);