Skip to content

Iss2#16

Open
MacdonEm wants to merge 8 commits into
masterfrom
iss2
Open

Iss2#16
MacdonEm wants to merge 8 commits into
masterfrom
iss2

Conversation

@MacdonEm

Copy link
Copy Markdown
Owner

As stated in the previous commit, the code smells a bit and probs needs to be 100% reorganized. I did not already do this because I think it needs to be 100% reorganized later anyway for one of the immediate features.

Issue 2, on the other hand, has been solved (with the added bonus of I really know how it all works, why it all works, and how it's doing it all, which is nice!

SCATT can now differentiate between the version 2 offline editor and the version 3.0.0 (The most recent version of Scratch to my current knowledge).

…anged Scatt.java and Report.java to use those directories instead.
… project. SCATT seems to work properly with version 2 Scratch projects.
… log for some reason, but I think I did it right this time. No actual code changes, just updated git.
…have to constantly delete and reupload them. Until the multiple submissions feature is up and running, moving files in an out of SCATT this way should prevent the thing from crashing like last time :)
… the same materials. It also works for the .sb2 (shocking) and does not for the .sb3 (crazy) but we are successfully in the right areas.
…ook was A LOT of testing to see what things did, about 2 hours worth of the wrong work (it's stashed), and another hour of trial and error. But I think I know how everything works, why it's working, and what it's producing now. So that's pretty cool :)
…tandpoint, but it can now differentiate between versions and provide the appropriate report (version 3 just supplies a bunch of zeroes for now)
@MacdonEm MacdonEm added the SCATT UPDATE Pull requests that solved issues or accomplished goals label Apr 27, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

SCATT UPDATE Pull requests that solved issues or accomplished goals

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant