Skip to content

DTT (Staging > Test)#9676

Merged
balderdash merged 26 commits into
testfrom
staging
Jul 22, 2016
Merged

DTT (Staging > Test)#9676
balderdash merged 26 commits into
testfrom
staging

Conversation

@balderdash

Copy link
Copy Markdown
Contributor

No description provided.

Josh Lory and others added 26 commits July 19, 2016 17:02
Limiting admin access to Section was making them unable to see progress/results in teacher dashboard.
…[test all]

When generating code directly from XML, we use a helper that initializes
a new readonly blockspace, populates it with blocks, and generates code
from there. We would formerly accomplish this by populating the user's
working blockspace with the blocks, generating the code, and then
replacing the user's original blockspace, but this had some problems.

Unfortunately, the code generator, when scanning the code for variables
whose declaration it wants to pull up into the top of the block, only
considers the blocks in the main block space when looking for those
variables. This can lead to problems, as with [course 4, stage 18][1]

The simple fix is to have the generator directly scan for variables in
the blocks that it's using.

[1]: https://studio.code.org/s/course4/stage/18/puzzle/2
Add initial `ask` block to prompt for user input in Play Lab
…eadonly-code

Allow for variable declarations in readonly-generated code
2280d07 Merge pull request #9610 from code-dot-org/ask-block (Josh Lory)
5ec4d1b Merge pull request #9667 from code-dot-org/allow-for-while-blocks-in-20hour (Elijah Hamovitz)
96af5fc Merge pull request #9672 from code-dot-org/revert-9632-encodeUrl (Mehal Shah)
6295268 Revert "[test ui] Encode URLs in free response" (Mehal Shah)
add photo credit for argentina partner's picture on applab page
@balderdash balderdash merged commit fc052a7 into test Jul 22, 2016
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.

6 participants