-
Notifications
You must be signed in to change notification settings - Fork 2
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
revise implementation-notes.md #66
Labels
Comments
Also recommended to add a Terminology section to implementation-notes.md. |
Check list of things that implementation-notes.md should contain or mention for this sim:
|
pixelzoom
added a commit
that referenced
this issue
Feb 26, 2017
Most complicated bits:
|
|
Model hierarchy for shopping screens:
|
pixelzoom
added a commit
that referenced
this issue
Mar 6, 2017
pixelzoom
added a commit
that referenced
this issue
Mar 6, 2017
pixelzoom
added a commit
that referenced
this issue
Mar 6, 2017
pixelzoom
added a commit
that referenced
this issue
Mar 6, 2017
pixelzoom
added a commit
that referenced
this issue
Mar 6, 2017
pixelzoom
added a commit
that referenced
this issue
Mar 7, 2017
pixelzoom
added a commit
that referenced
this issue
Mar 7, 2017
pixelzoom
added a commit
that referenced
this issue
Mar 7, 2017
pixelzoom
added a commit
that referenced
this issue
Mar 7, 2017
pixelzoom
added a commit
that referenced
this issue
Mar 7, 2017
pixelzoom
added a commit
that referenced
this issue
Mar 7, 2017
pixelzoom
added a commit
that referenced
this issue
Mar 7, 2017
Done, closing. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Code review #52.
Deficiencies in implementation-notes.md:
(1) Some of the entries would be more appropriate as GitHub issues, since they describe unresolved issues (e.g. potential memory leaks)
(2) There is a heading for the "Racing Lab" screen, but no information.
(3) Not much of an "overview" is gleaned by reading this, which is the main purpose of the file.
Whoever finishes off this sim should revisit and revise this file.
The text was updated successfully, but these errors were encountered: