javaee-samples / javaee7-hol

Java EE 7 Hands-on Lab
http://htmlpreview.github.io/?https://github.com/javaee-samples/javaee7-hol/blob/master/docs/javaee7-hol.html
Other
136 stars 92 forks source link

Added instructions for IntelliJ IDEA #1

Closed riy closed 10 years ago

riy commented 10 years ago

Hi Arun,

As promised today during the JavaEE 7 HOL at Devoxx here's my pull request for the IntelliJ IDEA setup instructions.

I assume you'd want to transform the file into your Pages/PDF file and also embed the screenshots. They're all in my Dropbox now - the URLs are in the document as well. I'm not familiar with AsciiDoc, otherwise I would have done something there but I hope this file helps for now.

Rias

arun-gupta commented 10 years ago

Thanks a lot!

I've started converting the docs to use AsciiDoc and will consume this over there. More details can be found at:

https://github.com/arun-gupta/javaee7-hol/issues/2

arun-gupta commented 10 years ago

Can you send an updated PR using AsciiDoc ?

I've created first draft of the instructions using AsciiDoc at docs/asciidoc.

riy commented 10 years ago

Done. I can't send a new PR though. Not sure why, but I think because my previous one is still not merged into your master branch. Can you do that so I can try again?

Or wait... maybe it's because I merged your master branch as an remote-tracking branch into mine.

Please check and let me know when to try again.

arun-gupta commented 10 years ago

There should be no need to approve the previous PR. You can just change the code, send an updated PR.

What is the error message are you getting otherwise ?

riy commented 10 years ago

I don't get an error message but don't see the buttons/fields I expect to see. Instead I see my old (open) pull request only.

I should see a field like this (different project, no open pull requests): http://d.pr/i/J8Zm But I see this: http://d.pr/i/qHI8

riy commented 10 years ago

Wait, when looking at this conversation I see that today's commits show up here. I guess this (old) pull request has automatically been updated with them. I assume you just have to pull it now. Since I've set your master branch as a remote-tracking branch and fetched your latest changes my branch should be up-to-date with your latest changes.

riy commented 10 years ago

:-)

arun-gupta commented 10 years ago

I may clean up some instructions to bring consistency, good work overall!

Thanks for the PR!

riy commented 10 years ago

Yw! Sure, go ahead. I tried to follow your file naming conventions, hope it all looks fine.

arun-gupta commented 10 years ago

I looked at your source and fixed bullets on all other chapters, looking much better.

Everything integrated now.

riy commented 10 years ago

Yes, headers and images break the list numbering... and then numbering re-starts from 1. Spent 90% of my time only on trying to fix that :-)

arun-gupta commented 10 years ago

Have you tried the lab steps ?

riy commented 10 years ago

Just that one time during your hands-on-lab at Devoxx. I think I completed it there as 6th or 7th.

arun-gupta commented 10 years ago

Cool, do you want to write a blog talking about your experience ?

I think it'll be a good story stating that you attended the Devoxx lab and became a contributor :-)

On Fri, Dec 6, 2013 at 12:25 PM, Rias A. Sherzad notifications@github.comwrote:

Just that one time during your hands-on-lab at Devoxx. I think I completed it there as 6th or 7th.

— Reply to this email directly or view it on GitHubhttps://github.com/javaee-samples/javaee7-hol/pull/1#issuecomment-30026296 .

http://blog.arungupta.me http://twitter.com/arungupta

riy commented 10 years ago

I can't right now :-/ I have 4 or 5 half-finished articles for my site (http://www.theserverside.de) waiting for me to finish them... :-( I will keep it in mind and get back to you if/when I find the time. I like the idea, also Devoxx and especially the HOLs were great!

arun-gupta commented 10 years ago

OK, good luck, thanks again!

riy commented 10 years ago

Thanks, good luck to you, too :-)