Uploaded image for project: 'IGB'
  1. IGB
  2. IGBF-2139

Modify App and Edit App page layouts

    Details

    • Type: Improvement
    • Status: Closed (View Workflow)
    • Priority: Major
    • Resolution: Done
    • Affects Version/s: None
    • Fix Version/s: None
    • Labels:
      None

      Description

      Change App page layout as shown in Slide 2 of:
      https://www.dropbox.com/s/7k8ue6ha3l1ggc5/apppage.pptx?dl=0.

      Summary:

      • Move App install button to position immediately underneath where Editor Actions menu appears when an editor is logged in.
      • Show App version above Categories (two places on the page, not just one)
      • Area on the page where Editor Actions menu appears is always either empty or is showing the Editor Actions menu. Goal is to provide an instant cue to user regarding their login status and permissions

        Attachments

          Activity

          Show
          noor91zahara Noor Zahara (Inactive) added a comment - Code changes - https://bitbucket.org/nzahara/appstore-local/branch/IGBF-2139#diff
          Hide
          ann.loraine Ann Loraine added a comment -

          Code looks fine functionally, but I have not seen it live. Moving to ready for PR anyway.

          Show
          ann.loraine Ann Loraine added a comment - Code looks fine functionally, but I have not seen it live. Moving to ready for PR anyway.
          Show
          noor91zahara Noor Zahara (Inactive) added a comment - PR Submitted - https://bitbucket.org/lorainelab/appstore/pull-requests/158/igbf-2139-layout-ui-changes-in-app-page/diff
          Hide
          ann.loraine Ann Loraine added a comment -

          Merged not yet deployed on dev-appstore-2.bioviz.org.

          Show
          ann.loraine Ann Loraine added a comment - Merged not yet deployed on dev-appstore-2.bioviz.org.
          Hide
          shamika Shamika Gajanan Kulkarni (Inactive) added a comment - - edited

          Tested the ticket. Observed that the following is working as expected.
          1) The App version is visible above Categories with font size as mentioned in pptx
          2) The App install button is positioned immediately underneath the Editor Actions
          3) The 'Go back to home' link is not visible anymore.
          4) When IGB 9.1 endpoint does not respond, the download symbol besides 'Get latest IGB' is visible, which when clicked redirects to 'https://bioviz.org'
          5) When IGB endpoint responds, 'Install this App' is visible along with the download symbol.
          6) On installing the app, the content changes to 'Installed' with a success checkmark
          7) The hyperlink from version compatibility has been removed

          However, there are two observations which did not satisfy what was mentioned in pptx
          1) If the app is installed, I was still able to see the 'Hand Pointer' on box-release-info content.
          2) If a new version of the app is released and the lower version of app is already installed, the content does not change to 'Update this app'. Please refer the screenshot attached.

          Kindly check Noor Zahara

          Show
          shamika Shamika Gajanan Kulkarni (Inactive) added a comment - - edited Tested the ticket. Observed that the following is working as expected. 1) The App version is visible above Categories with font size as mentioned in pptx 2) The App install button is positioned immediately underneath the Editor Actions 3) The 'Go back to home' link is not visible anymore. 4) When IGB 9.1 endpoint does not respond, the download symbol besides 'Get latest IGB' is visible, which when clicked redirects to 'https://bioviz.org' 5) When IGB endpoint responds, 'Install this App' is visible along with the download symbol. 6) On installing the app, the content changes to 'Installed' with a success checkmark 7) The hyperlink from version compatibility has been removed However, there are two observations which did not satisfy what was mentioned in pptx 1) If the app is installed, I was still able to see the 'Hand Pointer' on box-release-info content. 2) If a new version of the app is released and the lower version of app is already installed, the content does not change to 'Update this app'. Please refer the screenshot attached. Kindly check Noor Zahara
          Hide
          sameer Sameer Shanbhag (Inactive) added a comment - - edited

          1. Needs to be fixed. Noor Zahara Riddhi Jagdish Patil
          2. Known Bug and reported in IGBF-2149

          Show
          sameer Sameer Shanbhag (Inactive) added a comment - - edited 1. Needs to be fixed. Noor Zahara Riddhi Jagdish Patil 2. Known Bug and reported in IGBF-2149
          Hide
          shamika Shamika Gajanan Kulkarni (Inactive) added a comment -

          Prof. [~aloraine] The 'Update this app' feature works for App for Testing App Workflow and Get average score after refreshing App Manager, but is not working for Super Simple IGB App.

          Show
          shamika Shamika Gajanan Kulkarni (Inactive) added a comment - Prof. [~aloraine] The 'Update this app' feature works for App for Testing App Workflow and Get average score after refreshing App Manager, but is not working for Super Simple IGB App.
          Show
          rpatil14 Riddhi Jagdish Patil (Inactive) added a comment - Fixed issue 1: https://bitbucket.org/rpatil14/appstore_riddhi/branch/IGBF-2139-BugFix#diff cc. [~aloraine]
          Hide
          ann.loraine Ann Loraine added a comment -

          I have not tested this, but it looks like it will work. Please submit PR Riddhi Jagdish Patil

          Show
          ann.loraine Ann Loraine added a comment - I have not tested this, but it looks like it will work. Please submit PR Riddhi Jagdish Patil
          Show
          rpatil14 Riddhi Jagdish Patil (Inactive) added a comment - Pull request submitted: https://bitbucket.org/lorainelab/appstore/pull-requests/174/igbf-2139-show-not-allowed-cursor-for/diff
          Hide
          ann.loraine Ann Loraine added a comment -

          Merged not yet deployed on dev-appstore-2.

          Show
          ann.loraine Ann Loraine added a comment - Merged not yet deployed on dev-appstore-2.
          Hide
          ann.loraine Ann Loraine added a comment -

          Deployed to https://dev-appstore-2.bioviz.org and ready for testing.

          Show
          ann.loraine Ann Loraine added a comment - Deployed to https://dev-appstore-2.bioviz.org and ready for testing.
          Hide
          prutha Prutha Kulkarni (Inactive) added a comment -

          The app page layouts are changed according to the powerpoint slide no. 2. Everything change mentioned in the slide is implemented and tested on dev-appstore.
          Moving the ticket to DONE.

          Show
          prutha Prutha Kulkarni (Inactive) added a comment - The app page layouts are changed according to the powerpoint slide no. 2. Everything change mentioned in the slide is implemented and tested on dev-appstore. Moving the ticket to DONE.

            People

            • Assignee:
              rpatil14 Riddhi Jagdish Patil (Inactive)
              Reporter:
              ann.loraine Ann Loraine
            • Votes:
              0 Vote for this issue
              Watchers:
              6 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: