fix: ID prev link via rel=previous

Author: oliverroick

src/hooks/useStacSearch.test.ts

@@ -192,7 +192,7 @@ describe('useStacSearch', () => {
     expect(postPayload).toEqual(response.links[0].body);
   });
 
-  it('includes nextPage callback', async () => {
+  it('includes previousPage callback', async () => {
     const response = {
       links: [{
         rel: 'prev',
@@ -225,4 +225,38 @@ describe('useStacSearch', () => {
     expect(result.current.results).toEqual({ data: '12345' });
     expect(postPayload).toEqual(response.links[0].body);
   });
+
+  it('includes previousPage callback (previous edition)', async () => {
+    const response = {
+      links: [{
+        rel: 'previous',
+        type: 'application/geo+json',
+        method: 'POST',
+        href: 'https://example.com/stac/search',
+        body: {
+          limit: 25,
+          token: 'prev:abc123'
+        }
+      }]
+    };
+    fetch.mockResponseOnce(JSON.stringify(response));
+
+    const { result, waitForNextUpdate } = renderHook(
+      () => useStacSearch(stacApi)
+    );
+
+    act(() => result.current.setDateRangeTo('2022-05-17'));
+    act(() => result.current.submit());
+    await waitForNextUpdate(); // wait to set results
+    expect(result.current.results).toEqual(response);
+    expect(result.current.previousPage).toBeDefined();
+
+    fetch.mockResponseOnce(JSON.stringify({ data: '12345' }));
+    act(() => result.current.previousPage && result.current.previousPage());
+    await waitForNextUpdate();
+
+    const postPayload = parseRequestPayload(fetch.mock.calls[1][1]);
+    expect(result.current.results).toEqual({ data: '12345' });
+    expect(postPayload).toEqual(response.links[0].body);
+  });
 });

src/hooks/useStacSearch.ts

@@ -69,7 +69,8 @@ function useStacSearch(stacApi: StacApi): StacSearchHook {
         setNextPage(() => () => loadNewPage(nextPageLink));
       }
 
-      const previousPageLink = links.find(({ rel }) => rel === 'prev');
+      // Turns not all STAC APIs implement the spec correctly, some advertise the prev link as previous
+      const previousPageLink = links.find(({ rel }) => ['prev', 'previous'].includes(rel));
       if (previousPageLink) {
         setPreviousPage(() => () => loadNewPage(previousPageLink));
       }