Skip to content

Commit

Permalink
fix: remove ember initial data (#1101)
Browse files Browse the repository at this point in the history
* remove data from ember route on serve and bump node plugin

* changes
  • Loading branch information
knownasilya authored Jul 11, 2020
1 parent a7bee3a commit 8ce8c0e
Show file tree
Hide file tree
Showing 8 changed files with 85,247 additions and 695 deletions.
11 changes: 3 additions & 8 deletions apps/strider/dist-lib/routes/index.js

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion apps/strider/dist-lib/routes/index.js.map

Large diffs are not rendered by default.

85,883 changes: 85,221 additions & 662 deletions apps/strider/dist/scripts/app.js

Large diffs are not rendered by default.

16 changes: 3 additions & 13 deletions apps/strider/lib/routes/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -68,19 +68,9 @@ exports.emberIndex = function (req, res) {
}

if (req.query.ember) {
return jobs.latestJobs(req.user, true, function (err, jobs) {
const availableProviders = Object.keys(common.userConfigs.provider).map(
function (k) {
return common.userConfigs.provider[k];
}
);
// TODO: only set if dev
res.header(
'Cache-Control',
'private, no-cache, no-store, must-revalidate'
);
res.render('ember/index.html', { jobs, availableProviders });
});
// TODO: only set if dev
res.header('Cache-Control', 'private, no-cache, no-store, must-revalidate');
res.render('ember/index.html');
}
};

Expand Down
2 changes: 1 addition & 1 deletion apps/strider/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -110,7 +110,7 @@
"strider-heroku": "^0.2.0",
"strider-mailer": "^0.2.0",
"strider-metadata": "^0.0.3",
"strider-node": "^2.0.0",
"strider-node": "^2.0.1",
"strider-python": "^0.2.1",
"strider-ruby": "^0.0.2",
"strider-simple-runner": "^2.0.0",
Expand Down
5 changes: 1 addition & 4 deletions clients/strider-ui/app/index.html
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
<!DOCTYPE html>
<html>
<html lang="en-us">
<head>
<meta charset="utf-8" />
<meta http-equiv="X-UA-Compatible" content="IE=edge" />
Expand All @@ -18,9 +18,6 @@
<body>
{{content-for "body"}}

<script>
window.jobs = {{ jobs | scriptjson | raw }};
</script>
<script src="{{rootURL}}assets/vendor.js"></script>
<script src="{{rootURL}}assets/strider-ui.js"></script>

Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,11 @@
{
"changes": [
{
"packageName": "strider",
"comment": "Remove serve time data from ember index route and bump node plugin",
"type": "patch"
}
],
"packageName": "strider",
"email": "knownasilya@gmail.com"
}
12 changes: 6 additions & 6 deletions common/config/rush/pnpm-lock.yaml

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

1 comment on commit 8ce8c0e

@vercel
Copy link

@vercel vercel bot commented on 8ce8c0e Jul 11, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please sign in to comment.