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

Add gettingstarted.html #34

Closed
wants to merge 5 commits into from
Closed

Conversation

davelab6
Copy link
Contributor

New PR for samdroid-apps#10

Daksh added 5 commits January 24, 2016 10:13
Daksh, mechanically you have satisfied Sam's requests. But I am not convinced that the text you have written is very helpful to a newbie. For example, the discussion of the Frame should begin with What is the Frame? How do you access? Why do you want to access it? And much less important, How to change the settings.  You put settings upfront and only describe functionally the top left of the Frame.   I have similar comments for all of the other sections. Alas, more work needed.
@samdroid-apps
Copy link
Contributor

Wait, all of theme comments are from the previous pull request. Is this different from Daksh's patch?

@davelab6
Copy link
Contributor Author

This is @Daksh 's changeset, you see at the top it says,

davelab6 wants to merge 5 commits into sugarlabs:master from Daksh:getstarted

@davelab6
Copy link
Contributor Author

@Daksh will you be able to respond to Walter's comments?

@Daksh
Copy link

Daksh commented Aug 23, 2016

@davelab6 I have been away from this code for a long while now and away from programming totally due to academic demands from my school. I am sorry but I do not think that I would be able to help much over here for a long while

and I am sorry for the extremely late response

@davelab6
Copy link
Contributor Author

@Daksh thanks for following up :)

@quozl
Copy link
Contributor

quozl commented Sep 4, 2017

Closing because it will need to be rebased for #67.

@quozl quozl closed this Sep 4, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants