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

TOOLSDOC-566: Added OSO How To 'Custom Publish Your App' #286

Closed
wants to merge 2 commits into from

Conversation

@michellemurray
Copy link
Contributor

michellemurray commented Nov 24, 2014

@adietish : please review and leave comments for any suggestions/required changes.

*Ensure hidden application directories and files are visible*

. In the `Package Explorer` view, click the `View Menu` icon image:openshift_viewmenuicon.png[View Menu icon] and click `Filters`.
. From the table, ensure the `.* resources` check box is cleared and click `OK`.

This comment has been minimized.

Copy link
@adietish

adietish Dec 4, 2014

Member

You can also simply switch to the Navigator view where there's no filter that hides the .openshift folder.

*Create the post deploy action hook*

[start=3]
. In the `Package Explorer` view, expand `_{application name}_ > .openshift`.

This comment has been minimized.

Copy link
@adietish

adietish Dec 4, 2014

Member

Or switch to the Navigator view where no filter have to be modified.


[start=8]
. In the `Package Explorer` view, expand `_{application name}_ > .openshift > action_hooks`.
. Right-click the `post_deploy` file and click menu:Team[Add to Index].

This comment has been minimized.

Copy link
@adietish

adietish Dec 4, 2014

Member

This step can be skipped since we now present a Publish dialog which lists all files that are new/modified/deleted and then allows you to add the new files to the git index: https://issues.jboss.org/browse/JBIDE-10541

@adietish
Copy link
Member

adietish commented Dec 4, 2014

nice guide, thanks!
I commented where I thought I'd maybe do it differently.

@michellemurray
Copy link
Contributor Author

michellemurray commented Dec 5, 2014

Thanks for the great feedback @adietish . I have changed the info to the Navigator view as suggested and moved the info about Team > Add to Index to the Did You Know section. Please verify if you are happy with the changes so @xcoulon can merge the PR.

@xcoulon
Copy link
Member

xcoulon commented Dec 5, 2014

ok, waiting to @adietish approval, then I'll merge the PR.

@adietish
Copy link
Member

adietish commented Dec 5, 2014

+1, good to merge, nice documentation, thanks!

@xcoulon
Copy link
Member

xcoulon commented Dec 5, 2014

PR merged. Thanks, @michellemurray and @adietish !

@xcoulon xcoulon closed this Dec 5, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.