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

Update CONTRIBUTING.md #231

Merged
merged 3 commits into from
Jan 11, 2015
Merged

Update CONTRIBUTING.md #231

merged 3 commits into from
Jan 11, 2015

Conversation

ossanna16
Copy link

Hey Ola!

I was going through the tutorial on how to contribute and I noticed that it's not possible anymore to download the Gitbook editor. Instead you have to sign up on their website and open the book in your browser I guess. I actually couldn't do that and therefore couldn't get a new screenshot because I can't manage to download the tutorial to my computer via git :(. I always get the same error message:

$ git clone git@github.com:annaoz/organizer-manual.git
Cloning into 'organizer-manual' ...
Warning: Permanently added the RSA host key for IP address '192.30.252.130' to the list of known hosts.
Permission denied (publickey).
fatal: Could not read from remote repository.

Please make sure you have the correct access rights and the repository exists.

Either there's something wrong with the command or I'm making a mistake.

Hope it can be fixed!

Anna

@keikoro
Copy link

keikoro commented Jan 7, 2015

Hm, the repo for the editor still exists, as does this download page where it says the latest version is from autumn, don't the downloads there work anymore?

@keikoro
Copy link

keikoro commented Jan 7, 2015

Regarding cloning not working: could it be you're using an SSH key pair the public key of which you haven't added to GitHub yet? Have you already tried cloning via https instead of ssh?

I can clone fine from your fork as well as the original DG remote, so the repositories work alright.

You can check your ssh keys uploaded to GH here. On your HD, you'll find your keys in a hidden directory called ".ssh" in your home/user directory. The public key (ending in .pub) is the one you'll want to compare. To try cloning with https, go to the repo you want to clone on GH and toggle the git clone url displayed on the lower right-hand side (just click on https to get the https version of it).

Btw. it's always a good idea to create separate GitHub issues for separate problems (like the GitBook issue/pull request vs. the git clone issue here). Makes it easier to keep track of things. (:

@ossanna16
Copy link
Author

Thanks for the tips with the ssh keys. I'll look into that another time. I really only started contributing here a couple of days ago so I'm still learning :)

@ossanna16
Copy link
Author

The link to the dowload of the Gitbook editor doesn't work for me. It tells me "file not found". When I go to http://gitbook.com/editor it tells me "The desktop Editor is no longer supported and being replaced by this new web editor, source code is still available on GitHub." Hence I figured that one probably should use the new web editor instead of dowloading an old version of the Gitbook editor.

@keikoro
Copy link

keikoro commented Jan 7, 2015

I simply get redirected to the Gitbook website when I click the link to the editor you posted, I don't even get to see that note! But true, there's no word about the editor on their website anymore.

And I dunno why the download page on GitHub won't work for you, too bad. It's linked from the repository's README and contains downloads to a number of old versions. I reckon what they meant by not supporting the stand-alone editor anymore is that they won't add new features or provide support for it henceforth, not necessarily that one must/should ditch it right away. (I personally prefer native apps over web-/browser based ones, so I'm sad to see the stand-alone app go.)

@ossanna16
Copy link
Author

When I click the hyperlink in the sentence "Download the Gitbook Editor app to your computer." which is http://help.gitbook.io/editor/README.html I get the error message "404 File not found", I don't get redirected anywhere so either it only doesn't work for me which would be strange since I'm using Google Chrome and everything else works fine or the link needs to be updated.

Right, anyway it would be helpful to add one sentence that there are two options: Download the editor or use the web editor so people can choose what they are most comfortable working with.

@keikoro
Copy link

keikoro commented Jan 7, 2015

That's not the link I posted though! Check my comment above, I'm linking to the GitHub repo for the editor.

And yup, listing both options in the DG README sounds like a good idea. Maybe including a note that the stand-alone app is being phased out but can still be used, just so people don't get confused?

@ossanna16
Copy link
Author

I know that's not the link you sent but that's the link that is currently in the tutorial and that people will probably click on and then discover as well that it doesn't work that's why it needs to be changed.

That sounds good!

@keikoro
Copy link

keikoro commented Jan 8, 2015

Ahh, got it now!

@ossanna16
Copy link
Author

Thank you for helping me with the ssh key. I was able to generate and add one to GitHub and successfully cloned the repository :) What do you think about putting 1-2 sentences about ssh keys into the "Contribute" section? Since the tutorial is aimed at beginners I can imagine that other people that have never worked with GitHub before might run into the same problem.

@keikoro
Copy link

keikoro commented Jan 10, 2015

You're welcome!

About including it in Contribute: I'm not sure, but I think I personally wouldn't. Using git and GitHub works fine with https, which should be the default (which is why I was guessing you accidentally switched the git clone URLs to ssh at some point) and doesn't require changing any settings or an understanding of ssh. In the long run, I think ssh is preferable but if you don't really know what you're doing, it might create more confusion and problems than just sticking with https, especially as a beginner (e.g. accidental sharing of private key, forgetting about replicating the process on other machines or websites etc.).

Except maybe if the original DG tutorial already uses git pulling/pushing via ssh? Though in that case, I'd rather switch that to https.

@keimlink
Copy link
Contributor

GitHub has an excellent documentation on how to fork and clone a repository and how to create a pull request. So I would suggest to link to GitHub's documentation instead explaining it in the contribution guide.

@olasitarska
Copy link
Member

good point @keimlink !

@ossanna16
Copy link
Author

I updated the link to the download site for Gitbook editor (old one wasn't working), added a sentence and link that you can also use Gitbook in your browser and added a link to the documentation like Markus suggested.

@keikoro
Copy link

keikoro commented Jan 10, 2015

👍

asendecka added a commit that referenced this pull request Jan 11, 2015
@asendecka asendecka merged commit b97b299 into DjangoGirls:master Jan 11, 2015
@asendecka
Copy link
Contributor

Thank you @annaoz :) 🍰 😍. Thank you @kerstin and @keimlink!

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.

None yet

5 participants