Skip to content

Commit

Permalink
Merge branch 'release-next'
Browse files Browse the repository at this point in the history
  • Loading branch information
brophdawg11 committed Jan 13, 2023
2 parents 9953f73 + 529322d commit 92bc898
Show file tree
Hide file tree
Showing 38 changed files with 285 additions and 43 deletions.
2 changes: 1 addition & 1 deletion babel.config.js
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ module.exports = {
"@babel/preset-env",
{
targets: {
node: "current",
node: "14",
},
},
],
Expand Down
120 changes: 119 additions & 1 deletion integration/fetcher-test.ts
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
import { test } from "@playwright/test";
import { expect, test } from "@playwright/test";

import { createAppFixture, createFixture, js } from "./helpers/create-fixture";
import type { Fixture, AppFixture } from "./helpers/create-fixture";
Expand Down Expand Up @@ -143,6 +143,57 @@ test.describe("useFetcher", () => {
);
}
`,

"app/routes/fetcher-echo.jsx": js`
import { json } from "@remix-run/node";
import { useFetcher } from "@remix-run/react";
export async function action({ request }) {
await new Promise(r => setTimeout(r, 1000));
let value = (await request.formData()).get('value');
return json({ data: "ACTION " + value })
}
export async function loader({ request }) {
await new Promise(r => setTimeout(r, 1000));
let value = new URL(request.url).searchParams.get('value');
return json({ data: "LOADER " + value })
}
export default function Index() {
let fetcherValues = [];
if (typeof window !== 'undefined') {
if (!window.fetcherValues) {
window.fetcherValues = [];
}
fetcherValues = window.fetcherValues
}
let fetcher = useFetcher();
let currentValue = fetcher.state + '/' + fetcher.data?.data;
if (fetcherValues[fetcherValues.length - 1] !== currentValue) {
fetcherValues.push(currentValue)
}
return (
<>
<input id="fetcher-input" name="value" />
<button id="fetcher-load" onClick={() => {
let value = document.getElementById('fetcher-input').value;
fetcher.load('/fetcher-echo?value=' + value)
}}>Load</button>
<button id="fetcher-submit" onClick={() => {
let value = document.getElementById('fetcher-input').value;
fetcher.submit({ value }, { method: 'post', action: '/fetcher-echo' })
}}>Submit</button>
{fetcher.state === 'idle' ? <p id="fetcher-idle">IDLE</p> : null}
<pre>{JSON.stringify(fetcherValues)}</pre>
</>
);
}
`,
},
});

Expand Down Expand Up @@ -232,4 +283,71 @@ test.describe("useFetcher", () => {
await app.clickElement("#submit-index-post");
await page.waitForSelector(`pre:has-text("${PARENT_INDEX_ACTION}")`);
});

test("fetcher.load persists data through reloads", async ({ page }) => {
let app = new PlaywrightFixture(appFixture, page);

await app.goto("/fetcher-echo", true);
expect(await app.getHtml("pre")).toMatch(
JSON.stringify(["idle/undefined"])
);

await page.fill("#fetcher-input", "1");
await app.clickElement("#fetcher-load");
await page.waitForSelector("#fetcher-idle");
expect(await app.getHtml("pre")).toMatch(
JSON.stringify(["idle/undefined", "loading/undefined", "idle/LOADER 1"])
);

await page.fill("#fetcher-input", "2");
await app.clickElement("#fetcher-load");
await page.waitForSelector("#fetcher-idle");
expect(await app.getHtml("pre")).toMatch(
JSON.stringify([
"idle/undefined",
"loading/undefined",
"idle/LOADER 1",
"loading/LOADER 1", // Preserves old data during reload
"idle/LOADER 2",
])
);
});

test("fetcher.submit persists data through resubmissions", async ({
page,
}) => {
let app = new PlaywrightFixture(appFixture, page);

await app.goto("/fetcher-echo", true);
expect(await app.getHtml("pre")).toMatch(
JSON.stringify(["idle/undefined"])
);

await page.fill("#fetcher-input", "1");
await app.clickElement("#fetcher-submit");
await page.waitForSelector("#fetcher-idle");
expect(await app.getHtml("pre")).toMatch(
JSON.stringify([
"idle/undefined",
"submitting/undefined",
"loading/ACTION 1",
"idle/ACTION 1",
])
);

await page.fill("#fetcher-input", "2");
await app.clickElement("#fetcher-submit");
await page.waitForSelector("#fetcher-idle");
expect(await app.getHtml("pre")).toMatch(
JSON.stringify([
"idle/undefined",
"submitting/undefined",
"loading/ACTION 1",
"idle/ACTION 1",
"submitting/ACTION 1", // Preserves old data during resubmissions
"loading/ACTION 2",
"idle/ACTION 2",
])
);
});
});
7 changes: 7 additions & 0 deletions packages/create-remix/CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,5 +1,12 @@
# `create-remix`

## 1.10.1

### Patch Changes

- Updated dependencies:
- `@remix-run/[email protected]`

## 1.10.0

### Patch Changes
Expand Down
4 changes: 2 additions & 2 deletions packages/create-remix/package.json
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"name": "create-remix",
"version": "1.10.0",
"version": "1.10.1",
"description": "Create a new Remix app",
"homepage": "https://remix.run",
"bugs": {
Expand All @@ -17,7 +17,7 @@
"create-remix": "dist/cli.js"
},
"dependencies": {
"@remix-run/dev": "1.10.0"
"@remix-run/dev": "1.10.1"
},
"engines": {
"node": ">=14"
Expand Down
7 changes: 7 additions & 0 deletions packages/remix-architect/CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,5 +1,12 @@
# `@remix-run/architect`

## 1.10.1

### Patch Changes

- Updated dependencies:
- `@remix-run/[email protected]`

## 1.10.0

### Patch Changes
Expand Down
4 changes: 2 additions & 2 deletions packages/remix-architect/package.json
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"name": "@remix-run/architect",
"version": "1.10.0",
"version": "1.10.1",
"description": "Architect server request handler for Remix",
"bugs": {
"url": "https://github.com/remix-run/remix/issues"
Expand All @@ -15,7 +15,7 @@
"typings": "dist/index.d.ts",
"dependencies": {
"@architect/functions": "^5.0.2",
"@remix-run/node": "1.10.0",
"@remix-run/node": "1.10.1",
"@types/aws-lambda": "^8.10.82"
},
"devDependencies": {
Expand Down
7 changes: 7 additions & 0 deletions packages/remix-cloudflare-pages/CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,5 +1,12 @@
# `@remix-run/cloudflare-pages`

## 1.10.1

### Patch Changes

- Updated dependencies:
- `@remix-run/[email protected]`

## 1.10.0

### Patch Changes
Expand Down
4 changes: 2 additions & 2 deletions packages/remix-cloudflare-pages/package.json
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"name": "@remix-run/cloudflare-pages",
"version": "1.10.0",
"version": "1.10.1",
"description": "Cloudflare Pages request handler for Remix",
"bugs": {
"url": "https://github.com/remix-run/remix/issues"
Expand All @@ -15,7 +15,7 @@
"typings": "dist/index.d.ts",
"module": "dist/esm/index.js",
"dependencies": {
"@remix-run/cloudflare": "1.10.0"
"@remix-run/cloudflare": "1.10.1"
},
"devDependencies": {
"@cloudflare/workers-types": "^3.4.0",
Expand Down
7 changes: 7 additions & 0 deletions packages/remix-cloudflare-workers/CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,5 +1,12 @@
# `@remix-run/cloudflare-workers`

## 1.10.1

### Patch Changes

- Updated dependencies:
- `@remix-run/[email protected]`

## 1.10.0

### Patch Changes
Expand Down
4 changes: 2 additions & 2 deletions packages/remix-cloudflare-workers/package.json
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"name": "@remix-run/cloudflare-workers",
"version": "1.10.0",
"version": "1.10.1",
"description": "Cloudflare worker request handler for Remix",
"bugs": {
"url": "https://github.com/remix-run/remix/issues"
Expand All @@ -16,7 +16,7 @@
"module": "dist/esm/index.js",
"dependencies": {
"@cloudflare/kv-asset-handler": "^0.1.3",
"@remix-run/cloudflare": "1.10.0"
"@remix-run/cloudflare": "1.10.1"
},
"devDependencies": {
"@cloudflare/workers-types": "^3.4.0"
Expand Down
7 changes: 7 additions & 0 deletions packages/remix-cloudflare/CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,5 +1,12 @@
# `@remix-run/cloudflare`

## 1.10.1

### Patch Changes

- Updated dependencies:
- `@remix-run/[email protected]`

## 1.10.0

### Patch Changes
Expand Down
4 changes: 2 additions & 2 deletions packages/remix-cloudflare/package.json
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"name": "@remix-run/cloudflare",
"version": "1.10.0",
"version": "1.10.1",
"description": "Cloudflare platform abstractions for Remix",
"bugs": {
"url": "https://github.com/remix-run/remix/issues"
Expand All @@ -15,7 +15,7 @@
"typings": "dist/index.d.ts",
"dependencies": {
"@cloudflare/kv-asset-handler": "^0.1.3",
"@remix-run/server-runtime": "1.10.0"
"@remix-run/server-runtime": "1.10.1"
},
"devDependencies": {
"@cloudflare/workers-types": "^3.4.0"
Expand Down
7 changes: 7 additions & 0 deletions packages/remix-deno/CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,5 +1,12 @@
# `@remix-run/deno`

## 1.10.1

### Patch Changes

- Updated dependencies:
- `@remix-run/[email protected]`

## 1.10.0

### Patch Changes
Expand Down
4 changes: 2 additions & 2 deletions packages/remix-deno/package.json
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"name": "@remix-run/deno",
"version": "1.10.0",
"version": "1.10.1",
"description": "Deno platform abstractions for Remix",
"homepage": "https://remix.run",
"main": "./index.ts",
Expand All @@ -15,7 +15,7 @@
"license": "MIT",
"sideEffects": false,
"dependencies": {
"@remix-run/server-runtime": "1.10.0",
"@remix-run/server-runtime": "1.10.1",
"mime": "^3.0.0"
},
"engines": {
Expand Down
8 changes: 8 additions & 0 deletions packages/remix-dev/CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,5 +1,13 @@
# `@remix-run/dev`

## 1.10.1

### Patch Changes

- Update babel config to transpile down to node 14 ([#5047](https://github.com/remix-run/remix/pull/5047))
- Updated dependencies:
- `@remix-run/[email protected]`

## 1.10.0

### Patch Changes
Expand Down
8 changes: 4 additions & 4 deletions packages/remix-dev/package.json
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"name": "@remix-run/dev",
"version": "1.10.0",
"version": "1.10.1",
"description": "Dev tools and CLI for Remix",
"homepage": "https://remix.run",
"bugs": {
Expand All @@ -26,7 +26,7 @@
"@babel/types": "^7.20.2",
"@esbuild-plugins/node-modules-polyfill": "^0.1.4",
"@npmcli/package-json": "^2.0.0",
"@remix-run/server-runtime": "1.10.0",
"@remix-run/server-runtime": "1.10.1",
"arg": "^5.0.1",
"cacache": "^15.0.5",
"chalk": "^4.1.2",
Expand Down Expand Up @@ -63,7 +63,7 @@
"xdm": "^2.0.0"
},
"devDependencies": {
"@remix-run/serve": "1.10.0",
"@remix-run/serve": "1.10.1",
"@types/cacache": "^15.0.0",
"@types/gunzip-maybe": "^1.4.0",
"@types/inquirer": "^8.2.0",
Expand All @@ -82,7 +82,7 @@
"type-fest": "^2.16.0"
},
"peerDependencies": {
"@remix-run/serve": "^1.10.0-pre.0"
"@remix-run/serve": "^1.10.1-pre.0"
},
"peerDependenciesMeta": {
"@remix-run/serve": {
Expand Down
4 changes: 4 additions & 0 deletions packages/remix-eslint-config/CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,5 +1,9 @@
# `@remix-run/eslint-config`

## 1.10.1

No significant changes to this package were made in this release. [See the releases page on GitHub](https://github.com/remix-run/remix/releases/tag/remix%401.10.1) for an overview of all changes in v1.10.1.

## 1.10.0

No significant changes to this package were made in this release. [See the releases page on GitHub](https://github.com/remix-run/remix/releases/tag/remix%401.10.0) for an overview of all changes in v1.10.0.
Expand Down
2 changes: 1 addition & 1 deletion packages/remix-eslint-config/package.json
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"name": "@remix-run/eslint-config",
"version": "1.10.0",
"version": "1.10.1",
"description": "ESLint configuration for Remix projects",
"bugs": {
"url": "https://github.com/remix-run/remix/issues"
Expand Down
Loading

0 comments on commit 92bc898

Please sign in to comment.