-
Notifications
You must be signed in to change notification settings - Fork 167
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
che-docs #715 Adding documentation for Hosted Che #729
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hello @ibuziuk, what do you think of the proposed suggestions?
a24c111
to
0754410
Compare
d53dfc7
to
75ff493
Compare
@rkratky @supriya-bharadwaj @themr0c PR is ready for review |
|
||
[id='related-information-{context}'] | ||
== Related information | ||
NOTE: In order to use push the source code to GitHub from IDE you will need to generate SSH keys using the link:https://github.com/eclipse/che-theia/tree/master/plugins/ssh-plugin[SSH Plug-in] and upload the public key to your GitHub account (`Settings` -> `SSH and GPG keys` -> `New SSH key`) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There should be a section dedicated to this topic (c.f. the section in che-docs and the google doc for the draft ). I would rather add a link to that doc here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
good point
|
||
:context: {parent-context-of-hosted-che} | ||
Frequently Asked Questions (FAQs) and Troubleshooting can be found on the following link:https://github.com/redhat-developer/rh-che/blob/master/FAQ.adoc[link]. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We are centralizing all the issues/README/wiki to eclipse/che. Wouldn't make sense to move the hosted-che FAQ to eclipse/che as well?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
good point 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Really really good. Concise and clear. I have added a couple of comment but these are really minor stuff so I am ok to merge even with the current PR content.
@l0rd thank you for review - have added a link to "Configuring your VCS credentials for your workspaces" + moved FAQ / Troubleshooting to the doc. Once PR is merged and Hosted Che docs will be available we would also need to update the link in the UD footer - redhat-developer/rh-che#1479 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good job! 👍
@l0rd What is exactly the plan for the Hosted Che FAQ?
|
I will let @l0rd to comment but for me it does not make sense to Have Hosted Che docs in the https://github.com/eclipse/che wiki. I personally would really like to have all the Hosted Che docs consolidated in one place (including FAQ) and get rid of the https://github.com/redhat-developer/rh-che/blob/master/FAQ.adoc from the Downstream repo. |
I think there should be consistency. If we have upstream Che FAQ in the |
@l0rd, I agree about consistency, but I think that what you're suggesting actually makes things more confusing and inconsistent. What would make sense from the perspective of a user who doesn't know about our internal discussions and processes, is to have an FAQ doc in the repo to which it belongs. Putting Hosted Che FAQ to the upstream Eclipse Che repo is not intuitive. Upstream Che is an Eclipse Foundation project. Hosted Che is a Red Hat product. Just as we don't put CRW docs into the Eclipse Che repo, we shouldn't, in my opinion, do it with Hosted Che. |
I also think that putting Hosted Che FAQ [1] to upstream would be more confusing. However, Hosted Che FAQ bits are not part of the Che 7 GA endgame plan. So, I would propose for now:
After GA release we could reconsider what would be the best place for FAQ. [1] https://github.com/redhat-developer/rh-che/blob/master/FAQ.adoc |
+1 @ibuziuk |
Currently on Che community website there are direct links to hosted Che.
On Che community documentation there are direct links to hosted Che as well.
If we want to be consistent with what you are proposing we should move
those links away from upstream because, as you said, hosted Che is a Red
Hat product and we should separate community and downstream or user and
contributors will be confused.
However we are pushing for presenting hosted Che as the place to go and try
upstream Che (not CodeReady Workspaces). We are pushing to make upstream
and hosted Che based on the same identical (community) codebase. We are
considering deprecating rh-che GitHub repository.
I believe the latter is the right direction for us today (unification). But
I understand and find reasonable the first approach as well (separation).
But on both cases we need to be consistent.
…On Tue, Jul 9, 2019, 08:40 Robert Kratky ***@***.***> wrote:
+1 @ibuziuk <https://github.com/ibuziuk>
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#729>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAEUF34ZPNALK3RQHC4GLPDP6QXFBANCNFSM4HXHO46A>
.
|
Anyway +1 to merge this PR as it is and open an issue/question to discuss where the hosted-che FAQ should be located. |
@l0rd sounds good, I have updated FAQ related issue redhat-developer/rh-che#1479 accordingly |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I aggregated suggestions in https://github.com/ibuziuk/che-docs/pull/1/files, I will remove my flood and suggestions from here, and I apologize for the flood.
@rkratky will let you review first ;-) ^ |
Link |
@ibuziuk Yes I removed most images and replaced them with a textual description which should last longer. |
@themr0c most = all but one ? |
Yes, screenshots that are at the end of the procedure are fine. |
Co-Authored-By: Fabrice Flore-Thébault <ffloreth@redhat.com> Robert Kratky <rkratky@redhat.com> Signed-off-by: Ilya Buziuk <ibuziuk@redhat.com>
@themr0c PR is https://github.com/ibuziuk/che-docs/pull/1/files merged and all the changes are squashed in 1 commit. @rkratky waiting for your final approval before merging to master |
Signed-off-by: Ilya Buziuk ibuziuk@redhat.com
What does this PR do?
Adding documentation for Hosted Che
What issues does this PR fix or reference?
#715