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

          aloraine Ann Loraine created issue -
          aloraine Ann Loraine made changes -
          Field Original Value New Value
          Epic Link IGBF-1388 [ 17463 ]
          aloraine Ann Loraine made changes -
          Rank Ranked higher
          aloraine Ann Loraine made changes -
          Sprint Fall 7 : 11 Nov to 22 Nov [ 78 ]
          aloraine Ann Loraine made changes -
          Rank Ranked lower
          aloraine Ann Loraine made changes -
          Rank Ranked higher
          rpatil14 Riddhi Jagdish Patil (Inactive) made changes -
          Assignee Riddhi Jagdish Patil [ rpatil14 ]
          rpatil14 Riddhi Jagdish Patil (Inactive) made changes -
          Assignee Riddhi Jagdish Patil [ rpatil14 ]
          noor91zahara Noor Zahara made changes -
          Assignee Noor Zahara [ noor91zahara ]
          noor91zahara Noor Zahara made changes -
          Status To-Do [ 10305 ] In Progress [ 3 ]
          Show
          noor91zahara Noor Zahara added a comment - Code changes - https://bitbucket.org/nzahara/appstore-local/branch/IGBF-2139#diff
          noor91zahara Noor Zahara made changes -
          Status In Progress [ 3 ] Needs 1st Level Review [ 10005 ]
          noor91zahara Noor Zahara made changes -
          Assignee Noor Zahara [ noor91zahara ]
          aloraine Ann Loraine made changes -
          Status Needs 1st Level Review [ 10005 ] First Level Review in Progress [ 10301 ]
          Hide
          aloraine Ann Loraine added a comment -

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

          Show
          aloraine Ann Loraine added a comment - Code looks fine functionally, but I have not seen it live. Moving to ready for PR anyway.
          aloraine Ann Loraine made changes -
          Status First Level Review in Progress [ 10301 ] Ready for Pull Request [ 10304 ]
          aloraine Ann Loraine made changes -
          Assignee Noor Zahara [ noor91zahara ]
          Show
          noor91zahara Noor Zahara added a comment - PR Submitted - https://bitbucket.org/lorainelab/appstore/pull-requests/158/igbf-2139-layout-ui-changes-in-app-page/diff
          noor91zahara Noor Zahara made changes -
          Status Ready for Pull Request [ 10304 ] Pull Request Submitted [ 10101 ]
          noor91zahara Noor Zahara made changes -
          Assignee Noor Zahara [ noor91zahara ]
          aloraine Ann Loraine made changes -
          Assignee Ann Loraine [ aloraine ]
          aloraine Ann Loraine made changes -
          Status Pull Request Submitted [ 10101 ] Reviewing Pull Request [ 10303 ]
          aloraine Ann Loraine made changes -
          Status Reviewing Pull Request [ 10303 ] Merged Needs Testing [ 10002 ]
          Hide
          aloraine Ann Loraine added a comment -

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

          Show
          aloraine Ann Loraine added a comment - Merged not yet deployed on dev-appstore-2.bioviz.org.
          aloraine Ann Loraine made changes -
          Comment [ There are many instances of "latest" in the code, suggesting the business logic for display apps is using date not version. This should be fixed. Because this is a different issue, have created a new ticket for that - https://jira.transvar.org/browse/IGBF-2145
          ]
          aloraine Ann Loraine made changes -
          Assignee Ann Loraine [ aloraine ]
          shamika Shamika Gajanan Kulkarni (Inactive) made changes -
          Assignee Shamika Gajanan Kulkarni [ shamika ]
          shamika Shamika Gajanan Kulkarni (Inactive) made changes -
          Status Merged Needs Testing [ 10002 ] Post-merge Testing In Progress [ 10003 ]
          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
          shamika Shamika Gajanan Kulkarni (Inactive) made changes -
          Status Post-merge Testing In Progress [ 10003 ] To-Do [ 10305 ]
          shamika Shamika Gajanan Kulkarni (Inactive) made changes -
          Assignee Shamika Gajanan Kulkarni [ shamika ] Noor Zahara [ noor91zahara ]
          shamika Shamika Gajanan Kulkarni (Inactive) made changes -
          Attachment appstore1.JPG [ 14518 ]
          Attachment appstore2.JPG [ 14519 ]
          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. Ann Loraine 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. Ann Loraine 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.
          rpatil14 Riddhi Jagdish Patil (Inactive) made changes -
          Assignee Noor Zahara [ noor91zahara ] Riddhi Jagdish Patil [ rpatil14 ]
          rpatil14 Riddhi Jagdish Patil (Inactive) made changes -
          Status To-Do [ 10305 ] In Progress [ 3 ]
          Show
          rpatil14 Riddhi Jagdish Patil (Inactive) added a comment - Fixed issue 1: https://bitbucket.org/rpatil14/appstore_riddhi/branch/IGBF-2139-BugFix#diff cc. Ann Loraine
          rpatil14 Riddhi Jagdish Patil (Inactive) made changes -
          Status In Progress [ 3 ] Needs 1st Level Review [ 10005 ]
          rpatil14 Riddhi Jagdish Patil (Inactive) made changes -
          Assignee Riddhi Jagdish Patil [ rpatil14 ]
          aloraine Ann Loraine made changes -
          Sprint Fall 7 : 11 Nov to 22 Nov [ 78 ] Fall 7 : 11 Nov to 22 Nov, Fall 8 : 25 Nov to 6 Dec [ 78, 79 ]
          aloraine Ann Loraine made changes -
          Rank Ranked higher
          Hide
          aloraine Ann Loraine added a comment -

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

          Show
          aloraine Ann Loraine added a comment - I have not tested this, but it looks like it will work. Please submit PR Riddhi Jagdish Patil
          aloraine Ann Loraine made changes -
          Status Needs 1st Level Review [ 10005 ] First Level Review in Progress [ 10301 ]
          aloraine Ann Loraine made changes -
          Status First Level Review in Progress [ 10301 ] Ready for Pull Request [ 10304 ]
          aloraine Ann Loraine made changes -
          Assignee Riddhi Jagdish Patil [ rpatil14 ]
          rpatil14 Riddhi Jagdish Patil (Inactive) made changes -
          Status Ready for Pull Request [ 10304 ] Pull Request Submitted [ 10101 ]
          rpatil14 Riddhi Jagdish Patil (Inactive) made changes -
          Assignee Riddhi Jagdish Patil [ rpatil14 ]
          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
          aloraine Ann Loraine added a comment -

          Merged not yet deployed on dev-appstore-2.

          Show
          aloraine Ann Loraine added a comment - Merged not yet deployed on dev-appstore-2.
          aloraine Ann Loraine made changes -
          Status Pull Request Submitted [ 10101 ] Reviewing Pull Request [ 10303 ]
          aloraine Ann Loraine made changes -
          Status Reviewing Pull Request [ 10303 ] Merged Needs Testing [ 10002 ]
          Hide
          aloraine Ann Loraine added a comment -

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

          Show
          aloraine Ann Loraine added a comment - Deployed to https://dev-appstore-2.bioviz.org and ready for testing.
          prutha Prutha Kulkarni (Inactive) made changes -
          Assignee Prutha Kulkarni [ prutha ]
          prutha Prutha Kulkarni (Inactive) made changes -
          Status Merged Needs Testing [ 10002 ] Post-merge Testing In Progress [ 10003 ]
          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.
          prutha Prutha Kulkarni (Inactive) made changes -
          Assignee Prutha Kulkarni [ prutha ] Riddhi Jagdish Patil [ rpatil14 ]
          prutha Prutha Kulkarni (Inactive) made changes -
          Resolution Done [ 10000 ]
          Status Post-merge Testing In Progress [ 10003 ] Closed [ 6 ]

            People

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

              Dates

              • Created:
                Updated:
                Resolved: