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

che-docs #715 Adding documentation for Hosted Che #729

Merged
merged 1 commit into from
Jul 11, 2019

Conversation

ibuziuk
Copy link
Member

@ibuziuk ibuziuk commented Jun 12, 2019

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

Copy link
Contributor

@themr0c themr0c left a 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?

src/main/pages/che-7/overview/assembly_hosted-che.adoc Outdated Show resolved Hide resolved
src/main/pages/che-7/overview/assembly_hosted-che.adoc Outdated Show resolved Hide resolved
src/main/pages/che-7/overview/assembly_hosted-che.adoc Outdated Show resolved Hide resolved
src/main/pages/che-7/overview/assembly_hosted-che.adoc Outdated Show resolved Hide resolved
@l0rd l0rd mentioned this pull request Jun 27, 2019
85 tasks
@ibuziuk ibuziuk force-pushed the che-docs-715 branch 2 times, most recently from a24c111 to 0754410 Compare July 1, 2019 14:59
@ibuziuk ibuziuk force-pushed the che-docs-715 branch 3 times, most recently from d53dfc7 to 75ff493 Compare July 5, 2019 15:48
@ibuziuk ibuziuk requested a review from rkratky July 5, 2019 15:49
@ibuziuk ibuziuk marked this pull request as ready for review July 5, 2019 15:49
@ibuziuk ibuziuk requested a review from slemeur as a code owner July 5, 2019 15:49
@ibuziuk
Copy link
Member Author

ibuziuk commented Jul 5, 2019

@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`)
Copy link
Contributor

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.

Copy link
Member Author

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].
Copy link
Contributor

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?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

good point 👍

Copy link
Contributor

@l0rd l0rd left a 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.

@ibuziuk
Copy link
Member Author

ibuziuk commented Jul 8, 2019

@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

Copy link
Contributor

@Katka92 Katka92 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good job! 👍

@themr0c
Copy link
Contributor

themr0c commented Jul 8, 2019

@l0rd What is exactly the plan for the Hosted Che FAQ?

@ibuziuk
Copy link
Member Author

ibuziuk commented Jul 8, 2019

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.

@l0rd
Copy link
Contributor

l0rd commented Jul 8, 2019

I think there should be consistency. If we have upstream Che FAQ in the eclipse/che wiki we should have hosted Che FAQ in that same wiki as well. Or we can move both in che-docs. But it should be consistent.

@rkratky
Copy link

rkratky commented Jul 9, 2019

@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.

@ibuziuk
Copy link
Member Author

ibuziuk commented Jul 9, 2019

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:

[id='faq']
== Frequently Asked Questions (FAQs) and Troubleshooting
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].

After GA release we could reconsider what would be the best place for FAQ.
@rkratky @l0rd wdyt?

[1] https://github.com/redhat-developer/rh-che/blob/master/FAQ.adoc

@rkratky
Copy link

rkratky commented Jul 9, 2019

+1 @ibuziuk

@l0rd
Copy link
Contributor

l0rd commented Jul 9, 2019 via email

@l0rd
Copy link
Contributor

l0rd commented Jul 9, 2019

Anyway +1 to merge this PR as it is and open an issue/question to discuss where the hosted-che FAQ should be located.

@ibuziuk
Copy link
Member Author

ibuziuk commented Jul 9, 2019

@l0rd sounds good, I have updated FAQ related issue redhat-developer/rh-che#1479 accordingly

Copy link
Contributor

@themr0c themr0c left a 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.

@ibuziuk
Copy link
Member Author

ibuziuk commented Jul 9, 2019

@rkratky will let you review first ;-) ^

@ibuziuk
Copy link
Member Author

ibuziuk commented Jul 11, 2019

@rkratky @themr0c is it expected that all the images but one getting-started-devfile.png have been removed ?

@ibuziuk
Copy link
Member Author

ibuziuk commented Jul 11, 2019

Creating a worskpace from the user dashboard
This section describes how to create a workspace from the user dashboard in Hosted Che.

Prerequisites
Existing Hosted Che account. See Registering to Hosted Che.

Link Registering to Hosted Che. is broken

@themr0c
Copy link
Contributor

themr0c commented Jul 11, 2019

@ibuziuk Yes I removed most images and replaced them with a textual description which should last longer.

@ibuziuk
Copy link
Member Author

ibuziuk commented Jul 11, 2019

@themr0c most = all but one ?

@themr0c
Copy link
Contributor

themr0c commented Jul 11, 2019

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>
@ibuziuk
Copy link
Member Author

ibuziuk commented Jul 11, 2019

@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

@ibuziuk ibuziuk merged commit 9842296 into eclipse-che:master Jul 11, 2019
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.

6 participants