Skip to content
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

Asset Performance Fix #2174

Merged
merged 3 commits into from
Feb 5, 2019
Merged

Asset Performance Fix #2174

merged 3 commits into from
Feb 5, 2019

Conversation

wyattjoh
Copy link
Collaborator

@wyattjoh wyattjoh commented Feb 1, 2019

What does this PR do?

How do I test this PR?

  1. Load assets into your database (assuming you're running this on port 3000 in development mode):
let assets = [];
for (var i = 0; i < 75000; i++) {
  assets.push({url:"http://localhost:3000/dev/assets/title/" + i, closedAt:null,closedMessage:null,created_at: new ISODate(), id: "" + i + "", metadata: {}, scraped: null, settings: {}, tags: [], type: "assets", updated_at: new ISODate()});
}
db.assets.insert(assets);
  1. Load the settings page, notice that the page loads! (Note, there will be lots of missing data there, title, publication date, but that's ok!)

@cvle cvle merged commit 94786c2 into master Feb 5, 2019
@cvle cvle deleted the assets-perf branch February 5, 2019 21:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Admin Stories /api/v1/assets request returns 500 error for large JSON response sizes
3 participants