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

Port into Valkyrie the nice Help support of Spring RCP #10

Closed
bravegag opened this issue May 10, 2013 · 4 comments
Closed

Port into Valkyrie the nice Help support of Spring RCP #10

bravegag opened this issue May 10, 2013 · 4 comments

Comments

@bravegag
Copy link
Contributor

Hello,

The nice quick support for Help documentation was apparently not ported into Valkyrie. I mean the implementation of org.springframework.richclient.command.support.HelpContentsCommand.

The configuration in the original Spring RCP implementation was:


This by convention would look for the appropriate files and create a nice accessible help documentation.

PS: I'm interested in contributing to the project, I need to build a new GUI from scratch on top of a good architecture and Valkyrie is the best I could find. Let me know whether is ok to get write access.

@cmadsen
Copy link
Collaborator

cmadsen commented May 10, 2013

Any help is appreciated.

I would prefer to take in changes as pull requests?

@bravegag
Copy link
Contributor Author

How does it work with pull requests? is it like branching in subversion? what exactly I have to do? Thank you.

Can I not simply send patches instead?

@cmadsen
Copy link
Collaborator

cmadsen commented May 10, 2013

Fork Valkyrie-RCP via github.

Checkout/clone your fork and do the changes and commit them back to the fork at github.

Then create a pull request (using github) as described here https://help.github.com/articles/creating-a-pull-request and here https://help.github.com/articles/using-pull-requests

Once I receive the pull request I will review it and merge it into Valkyrie-RPC.

@bravegag
Copy link
Contributor Author

OK I got the HelpContents implemented up and running as in Spring RCP. I also integrated it into the simple sample and works. I will find time after lunch to go through the fork/merge back process.
helpisback

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

No branches or pull requests

2 participants