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.

Sign up for GitHub

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

Jump to bottom

cloning-instructions #17

Merged
merged 7 commits into from
Jan 23, 2017
Merged

cloning-instructions #17

merged 7 commits into from
Jan 23, 2017

Conversation

tracey-le
Copy link
Collaborator

Adding how to run the project in README.md

The images are a bit big and some of the spacing is off but here is the
edited draft from #14

lyneca and others added 3 commits January 22, 2017 10:21
@lyneca
add clone instructions to README.md
c2078eb
@tracey-le
Some extra notes — create db is for first time running, and note on h…
221351e 
…ow to view at localhost:8888
@tracey-le
Added how to run project with screenshots
0abc040 
The images are a bit big and some of the spacing is off but here is the
edited draft from #14
@tracey-le tracey-le changed the title Pr/15 cloning-instructions Jan 22, 2017
@tracey-le tracey-le mentioned this pull request Jan 22, 2017
Closed
tracey-le and others added 2 commits January 22, 2017 14:09
@tracey-le
Removed extra divider causing unwanted formatting of a heading
5589b66 
The line 'Only clone the repository and create the database when running for first time' was not meant to be a heading
@tracey-le
Replacing steps using + in Desktop with Clone/download button in GitHub
147317b 
Updated with a screenshot snippet
@tracey-le
Copy link
Collaborator Author

tracey-le commented Jan 22, 2017 •
edited
Loading

Will the tutors be able to do the pull request for the latest version of the branch pr/15? I don't seem to have the option to request a later version after updating pr/15 hmm

@tracey-le
Additional heading for separation and small grammar change
31d749e 
Separated running and opening parts with headings, and just split a
long sentence (part on cloning and opening in Desktop)
@juliawong
Copy link
Contributor

@t1-tracey Merged #15

Tip: if you want to link to a PR or an issue in a comment, just type # and then the number.

@tracey-le
Copy link
Collaborator Author

ooh okay thank you for the tip @juliawong 😄 that will make it a lot easier :)

thank you for all the merging! ☺️

@tracey-le
Copy link
Collaborator Author

tracey-le commented Jan 22, 2017 •
edited
Loading

@juliawong do you think you would be able to do the pull requests for #17 (this one on README.md, I added onto #15) and #16 (it has the live error messages)? 😊

@juliawong
Copy link
Contributor

I'm currently out but perhaps @liamcal or @ekohilas can help. Otherwise I'll take a look when I get back tonight.

So awesome to see you keep working on the project! :D

@tracey-le
Copy link
Collaborator Author

oh sorry, don't worry that's okay 😊

I'm so glad to get to still get in touch with you guys 😄😊

@juliawong
Copy link
Contributor

No need to apologise! :) If none of us get around to it feel free to message us here or on FB.

@tracey-le
Update README.md with small change
77cb889 
Tried to make clearer the step for going to GitHub Desktop page and downloading
@juliawong juliawong merged commit e6022d7 into master Jan 23, 2017
@tracey-le
Copy link
Collaborator Author

Thank you so much Julia! 😃😄

@tracey-le
Copy link
Collaborator Author

tracey-le commented Jan 23, 2017 •
edited
Loading

@lyneca do we still need this branch pr/15 (the how-to for README.md)? we were thinking about deleting a few branches to reduce confusion for #24. but of course always okay to leave this! :D

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@juliawong juliawong juliawong approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@tracey-le @juliawong @lyneca