Skip to content
This repository has been archived by the owner on Jan 11, 2023. It is now read-only.

Use most current manifest_data #713

Merged
merged 1 commit into from
Jul 28, 2019
Merged

Use most current manifest_data #713

merged 1 commit into from
Jul 28, 2019

Conversation

sanderhahn
Copy link
Contributor

Sapper build seemed one step behind changes, this PR intends to fix it.

Example of trouble experienced:

$ touch src/routes/xyz.svelte
$ ls __sapper__/dev/client/xyz*
ls: __sapper__/dev/client/xyz*: No such file or directory

$ touch src/routes/xyz2.svelte
$ ls __sapper__/dev/client/xyz*
__sapper__/dev/client/xyz.a25908c7.js

$ touch src/routes/xyz3.svelte
$ ls __sapper__/dev/client/xyz*
__sapper__/dev/client/xyz.a25908c7.js
__sapper__/dev/client/xyz2.02d1ee4f.js

@Conduitry
Copy link
Member

Does this also fix stuff like #611?

@sanderhahn
Copy link
Contributor Author

This PR does fix the following, which hopefully would be similar to issue #611:

$ echo 'export function get(req, res) { res.end("Hello\n"); }' >src/routes/hello.txt.js
$ curl -i "http://localhost:3000/hello.txt"
HTTP/1.1 404 Not Found

$ touch src/routes/x.svelte
$ curl -i 'http://localhost:3000/hello.txt'
HTTP/1.1 200 OK

@Conduitry Conduitry merged commit 4401613 into sveltejs:master Jul 28, 2019
@Conduitry
Copy link
Member

Released in 0.27.5.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants