Maven Site Plugin
  1. Maven Site Plugin
  2. MSITE-158

Review and revise plugin documentation

    Details

    • Type: Task Task
    • Status: Closed Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: None
    • Labels:
      None
    • Number of attachments :
      0

      Activity

      Hide
      Maria Odea Ching added a comment -

      Review comments by Vincent Siveton via the dev mailing list (subject: Please review site plugin documentation)

      ------
      Here is my comments:

      usage.html
      note => Bold font
      add a link to the Jetty server: http://jetty.mortbay.org/jetty/index.html

      sitedescriptor.html
      "For more references of the site descriptor, here's a link
      http://maven.apache.org/site.html" should be in the introduction.

      http://maven.apache.org/site.html is out of date thus to specify it.
      (IMHO we need a new page for this, maybe something like
      http://maven.apache.org/ref/current/maven-model/maven.html)

      "Site Descriptor" part should be renamed to "Publish Date"

      templatefile.html
      add a link to Velocity: http://jakarta.apache.org/velocity/

      moduleexcludes.html
      empty <excludeModules/>

      creatingskins.html
      add an example or a link to
      http://svn.apache.org/viewvc/maven/skins/trunk/maven-classic-skin/

      General comment
      i18n link is missing.
      Moreover, i18n.xml should be updated with the new bundle.

      Cheers,

      Vincent

      Show
      Maria Odea Ching added a comment - Review comments by Vincent Siveton via the dev mailing list (subject: Please review site plugin documentation) ------ Here is my comments: usage.html note => Bold font add a link to the Jetty server: http://jetty.mortbay.org/jetty/index.html sitedescriptor.html "For more references of the site descriptor, here's a link http://maven.apache.org/site.html " should be in the introduction. http://maven.apache.org/site.html is out of date thus to specify it. (IMHO we need a new page for this, maybe something like http://maven.apache.org/ref/current/maven-model/maven.html ) "Site Descriptor" part should be renamed to "Publish Date" templatefile.html add a link to Velocity: http://jakarta.apache.org/velocity/ moduleexcludes.html empty <excludeModules/> creatingskins.html add an example or a link to http://svn.apache.org/viewvc/maven/skins/trunk/maven-classic-skin/ General comment i18n link is missing. Moreover, i18n.xml should be updated with the new bundle. Cheers, Vincent
      Hide
      Vincent Siveton added a comment -

      Sorry but I dont see the i18 page in the navbar... (SVN rev 426241)

      Show
      Vincent Siveton added a comment - Sorry but I dont see the i18 page in the navbar... (SVN rev 426241)
      Hide
      Maria Odea Ching added a comment -

      Sorry I forgot to add the i18n link in the nav bar. My bad..
      I'll update the site.xml. Thanks

      Show
      Maria Odea Ching added a comment - Sorry I forgot to add the i18n link in the nav bar. My bad.. I'll update the site.xml. Thanks
      Hide
      Maria Odea Ching added a comment -

      I've already updated the site.xml to include the Internationalization doc.
      Please see http://people.apache.org/~oching/maven-site-plugin

      Thanks

      Show
      Maria Odea Ching added a comment - I've already updated the site.xml to include the Internationalization doc. Please see http://people.apache.org/~oching/maven-site-plugin Thanks
      Hide
      Vincent Siveton added a comment -

      Thanks Maria!

      Moreover, could you review my previous post? Some comments seem to be not applied...

      Show
      Vincent Siveton added a comment - Thanks Maria! Moreover, could you review my previous post? Some comments seem to be not applied...
      Hide
      Maria Odea Ching added a comment -

      Hi Vincent,

      I've reviewed your previous post and updated the docs and the staging site.

      Thanks

      Show
      Maria Odea Ching added a comment - Hi Vincent, I've reviewed your previous post and updated the docs and the staging site. Thanks

        People

        • Assignee:
          Maria Odea Ching
          Reporter:
          Maria Odea Ching
        • Votes:
          0 Vote for this issue
          Watchers:
          0 Start watching this issue

          Dates

          • Due:
            Created:
            Updated:
            Resolved:

            Time Tracking

            Estimated:
            Original Estimate - 1 day, 9 hours
            1d 9h
            Remaining:
            Time Spent - 1 day, 6 hours Remaining Estimate - 3 hours
            3h
            Logged:
            Time Spent - 1 day, 6 hours Remaining Estimate - 3 hours
            1d 6h