-
Notifications
You must be signed in to change notification settings - Fork 77
REPL now defaults to trying to use Svelte v3 #406
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
And then while we're at it, we should update the github oauth app to use v2.svelte.dev instead of svelte.technology. |
I think this takes care of the issue with what version of the compiler the REPL gets from unpkg: diff --git a/src/routes/repl/index.html b/src/routes/repl/index.html
index 4975e83..de5a0a3 100644
--- a/src/routes/repl/index.html
+++ b/src/routes/repl/index.html
@@ -248,7 +248,7 @@
return {
examples,
- version: query.version || 'latest',
+ version: query.version || '2',
components: [],
data: {},
@@ -277,7 +277,7 @@
url: ({ version, gist, query, demo }) => {
const params = [];
- if (version !== 'latest') params.push(`version=${version}`);
+ if (version !== '2') params.push(`version=${version}`);
if (gist) params.push(`gist=${gist.id}`);
else if (demo) params.push(`demo=${demo}`);
else if (query.gist) params.push(`gist=${query.gist}`); |
Eesh we're also going to have to do something about the template that's used in the REPL for downloaded zips. Right now, |
It might actually be best to just check an old v2 version of the template into this repo, because that's not really going to be getting updated any more. |
Fixed. |
Now that
svelte@latest
is v3, people visiting the v2 REPL without aversion
in their URL will get v3, which breaks everything. We should be explicit about fetching version 2 from unpkg.The text was updated successfully, but these errors were encountered: