Geronimo
  1. Geronimo
  2. GERONIMO-5274

Deploy New/Install New Application portlet needs some love to help out newbies

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.1.5, 2.2
    • Fix Version/s: 2.2.1
    • Component/s: console
    • Security Level: public (Regular issues)
    • Labels:
      None

      Description

      Got feedback from a new Geronimo user (one of our OpenJPA committers) when she tried to setup Geronimo 2.1.5 + OpenJPA2 + Daytrader 2.1.3.

      1) Title of "Deploy New" item in the navigation tree should be more generic, like "Deploy Applications" or "Deployer", as it can be used to redeploy apps
      2) The Portlet title of "Install New Applications" should be more generic just like the menu title, like "Deploy Applications", as it can also be used to redeploy apps
      3) The "Plan" field needs to be relabeled as "Plan (optional):" as the deployment plan is optional.
      4) The Help for the portlet needs some updating:
      4a) The Archive field can be WAR/EAR/RAR and not just the mentioned WAR type.
      4b) The Plan field needs more description, like mentioning it is optional. Also it should mention "archive" instead of "war" as multiple file types are supported as stated in 4a. Need to also reword that it can be a deployment plan not included in the archive or a stand-alone deployment plan when a RAR is being used or for providing environment specific settings like the DB plans used with Daytrader.
      4c) the "Redeploy application" option in the portlet is not mentioned in the Help and needs to be added
      5) we shouldn't use "app" as an abbreviation for "application" for the checkbox "Start app after install"

      1. 5274_2.1_en&zh.patch
        12 kB
        Di Hu
      2. 5274_2.2_en.patch
        6 kB
        Chi Runhua
      3. 5274_2.2_zh_CN.patch
        4 kB
        Chi Runhua

        Activity

        Hide
        Shawn Jiang added a comment -

        the last patch was applied to 21 branch. closing it.

        Show
        Shawn Jiang added a comment - the last patch was applied to 21 branch. closing it.
        Hide
        Di Hu added a comment -

        Attach a patch for branch/2.1 with both english and chinese descriptions update.

        Show
        Di Hu added a comment - Attach a patch for branch/2.1 with both english and chinese descriptions update.
        Hide
        LiWenQin added a comment - - edited

        The two patches referring to JIRA-5274 described above works well.

        However, there is a new problem I have occured:
        The LEFT "Console navigation" is always displayed in Chinese, although the browser language has been set to English.
        ATTENTION Please: I have already changed mylocale to en_US in the browser.

        I have tried it in IE and Firefox browser, the both have the problem above.
        The LEFT "Console navigation" cannot display English. At the same time, the RIGHT page can display the language according to the browser language.

        Show
        LiWenQin added a comment - - edited The two patches referring to JIRA-5274 described above works well. However, there is a new problem I have occured: The LEFT "Console navigation" is always displayed in Chinese, although the browser language has been set to English. ATTENTION Please: I have already changed mylocale to en_US in the browser. I have tried it in IE and Firefox browser, the both have the problem above. The LEFT "Console navigation" cannot display English. At the same time, the RIGHT page can display the language according to the browser language.
        Hide
        Shawn Jiang added a comment -

        zh_CN Patch committed to 22 and trunk. Thanks Jeff for the patch !

        Show
        Shawn Jiang added a comment - zh_CN Patch committed to 22 and trunk. Thanks Jeff for the patch !
        Hide
        Chi Runhua added a comment -

        Simp.Chinese update for branch/trunk

        Show
        Chi Runhua added a comment - Simp.Chinese update for branch/trunk
        Hide
        Shawn Jiang added a comment -

        Patch committed to 22 and trunk. Thanks Jeff for the patch !

        Show
        Shawn Jiang added a comment - Patch committed to 22 and trunk. Thanks Jeff for the patch !
        Hide
        Donald Woods added a comment -

        Looks good. Thanks.

        Show
        Donald Woods added a comment - Looks good. Thanks.
        Hide
        Chi Runhua added a comment -

        Attach a patch for branch/2.2 with english descriptions update only. The patch is ready for review.

        Show
        Chi Runhua added a comment - Attach a patch for branch/2.2 with english descriptions update only. The patch is ready for review.
        Hide
        Chi Runhua added a comment -

        Thank you very much for the feedback.

        I updated the descriptions and portlet infos based on the comments. Help messages are also updated accordingly.

        I'm attaching a patch for branch/2.2 for review. If the patch is accepted, I'll provide the Simpl.Chinese version of the update as well as for branch/2.1 and trunk.

        Jeff

        Show
        Chi Runhua added a comment - Thank you very much for the feedback. I updated the descriptions and portlet infos based on the comments. Help messages are also updated accordingly. I'm attaching a patch for branch/2.2 for review. If the patch is accepted, I'll provide the Simpl.Chinese version of the update as well as for branch/2.1 and trunk. Jeff

          People

          • Assignee:
            Shawn Jiang
            Reporter:
            Donald Woods
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development