Skip to content

Remove level lists from script edit page#10748

Merged
balderdash merged 1 commit into
stagingfrom
remove-level-lists
Sep 20, 2016
Merged

Remove level lists from script edit page#10748
balderdash merged 1 commit into
stagingfrom
remove-level-lists

Conversation

@balderdash

@balderdash balderdash commented Sep 20, 2016

Copy link
Copy Markdown
Contributor

Levelbuilders don't use this feature, and without it the page is served about 80x faster.

@balderdash

balderdash commented Sep 20, 2016

Copy link
Copy Markdown
Contributor Author

Diff ignoring whitespace is clearer: https://github.com/code-dot-org/code-dot-org/pull/10748/files?w=1

@joshlory

Copy link
Copy Markdown
Contributor

LGTM! Ship it!

@balderdash balderdash merged commit 7aa1ac7 into staging Sep 20, 2016
@balderdash balderdash deleted the remove-level-lists branch September 29, 2016 01:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants