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

Fix: Clicking "i" icon on App submission confirmation page does nothing

        Details

        • Type: Bug
        • Status: Closed (View Workflow)
        • Priority: Major
        • Resolution: Done
        • Affects Version/s: None
        • Fix Version/s: None
        • Labels:
          None
        • Story Points:
          1
        • Epic Link:
        • Sprint:
          Summer 3: 6 Jul - 17 Jul, Summer 4: 14 Jul - 28 Jul, Summer 5: 3 Aug - 14 Aug

          Description

          When a user submits an App, a confirmation page appears with three "i" icons.

          Clicking each "i" icon is supposed to open a tooltip with further information.

          Currently, this is not working.

          For this task, investigate why this is failing and fix the problem.

            Attachments

              Activity

              Ascending order - Click to sort in descending order
              ann.loraine Ann Loraine created issue -
              ann.loraine Ann Loraine made changes -
              Field Original Value New Value
              Epic Link IGBF-1388 [ 17463 ]
              ann.loraine Ann Loraine made changes -
              Rank Ranked higher
              ann.loraine Ann Loraine made changes -
              Comment [ Hello [~aloraine] ]
              pbadzuh Philip Badzuh (Inactive) made changes -
              Assignee Philip Badzuh [ pbadzuh ]
              pbadzuh Philip Badzuh (Inactive) made changes -
              Status To-Do [ 10305 ] In Progress [ 3 ]
              ann.loraine Ann Loraine made changes -
              Sprint Summer 3: 6 Jul - 17 Jul [ 98 ] Summer 3: 6 Jul - 17 Jul, Summer 4: 14 Jul - 28 Jul [ 98, 99 ]
              ann.loraine Ann Loraine made changes -
              Rank Ranked higher
              Hide
              Permalink
              pbadzuh Philip Badzuh (Inactive) added a comment - - edited

              Please see my changes below:
              https://bitbucket.org/pbadzuh/appstore_pbdev/branch/IGBF-2469#diff

              • issue: the tooltip JS was calling a function from a script that wasn't imported to generate its text content
              • solution: import the required script
              Show
              pbadzuh Philip Badzuh (Inactive) added a comment - - edited Please see my changes below: https://bitbucket.org/pbadzuh/appstore_pbdev/branch/IGBF-2469#diff issue: the tooltip JS was calling a function from a script that wasn't imported to generate its text content solution: import the required script
              pbadzuh Philip Badzuh (Inactive) made changes -
              Status In Progress [ 3 ] Needs 1st Level Review [ 10005 ]
              pbadzuh Philip Badzuh (Inactive) made changes -
              Assignee Philip Badzuh [ pbadzuh ]
              ann.loraine Ann Loraine made changes -
              Status Needs 1st Level Review [ 10005 ] First Level Review in Progress [ 10301 ]
              ann.loraine Ann Loraine made changes -
              Status First Level Review in Progress [ 10301 ] Ready for Pull Request [ 10304 ]
              ann.loraine Ann Loraine made changes -
              Assignee Philip Badzuh [ pbadzuh ]
              Hide
              Permalink
              ann.loraine Ann Loraine added a comment -

              I have not tested the code, but the change looks good. Please submit PR when ready.

              Show
              ann.loraine Ann Loraine added a comment - I have not tested the code, but the change looks good. Please submit PR when ready.
              Hide
              Permalink
              pbadzuh Philip Badzuh (Inactive) added a comment -

              Please see the PR below:
              https://bitbucket.org/lorainelab/appstore/pull-requests/215/igbf-2469-fix-i-icon-on-app-submission/diff

              Show
              pbadzuh Philip Badzuh (Inactive) added a comment - Please see the PR below: https://bitbucket.org/lorainelab/appstore/pull-requests/215/igbf-2469-fix-i-icon-on-app-submission/diff
              pbadzuh Philip Badzuh (Inactive) made changes -
              Status Ready for Pull Request [ 10304 ] Pull Request Submitted [ 10101 ]
              pbadzuh Philip Badzuh (Inactive) made changes -
              Assignee Philip Badzuh [ pbadzuh ]
              ann.loraine Ann Loraine made changes -
              Status Pull Request Submitted [ 10101 ] Reviewing Pull Request [ 10303 ]
              ann.loraine Ann Loraine made changes -
              Status Reviewing Pull Request [ 10303 ] Merged Needs Testing [ 10002 ]
              gchamma Jay Chamma (Inactive) made changes -
              Status Merged Needs Testing [ 10002 ] Post-merge Testing In Progress [ 10003 ]
              gchamma Jay Chamma (Inactive) made changes -
              Status Post-merge Testing In Progress [ 10003 ] Merged Needs Testing [ 10002 ]
              Hide
              Permalink
              ann.loraine Ann Loraine added a comment -

              Merged to the master branch. Updated app store code deployed to https://testappstore.bioviz.org and ready for testing.

              Show
              ann.loraine Ann Loraine added a comment - Merged to the master branch. Updated app store code deployed to https://testappstore.bioviz.org and ready for testing.
              Hide
              Permalink
              ann.loraine Ann Loraine added a comment - - edited

              Test on latest:

              • Chrome
              • Edge
              • Firefox
              • Safari
              Show
              ann.loraine Ann Loraine added a comment - - edited Test on latest: Chrome Edge Firefox Safari
              Hide
              Permalink
              ann.loraine Ann Loraine added a comment - - edited

              Test on https://testappstore.bioviz.org.

              Show
              ann.loraine Ann Loraine added a comment - - edited Test on https://testappstore.bioviz.org .
              chaitanya Chaitanya Kintali (Inactive) made changes -
              Assignee Chaitanya Kintali [ chaitanya ]
              chaitanya Chaitanya Kintali (Inactive) made changes -
              Status Merged Needs Testing [ 10002 ] Post-merge Testing In Progress [ 10003 ]
              Hide
              Permalink
              chaitanya Chaitanya Kintali (Inactive) added a comment - - edited

              Tested the issue in Chrome, Edge, Safari, Firefox, Opera and working as expected.

              Steps to follow.
              1) visit testappstore.bioviz.org
              2) submit an app
              3) click on the tooltip to see the message.

              Moving to Done.

              Show
              chaitanya Chaitanya Kintali (Inactive) added a comment - - edited Tested the issue in Chrome, Edge, Safari, Firefox, Opera and working as expected. Steps to follow. 1) visit testappstore.bioviz.org 2) submit an app 3) click on the tooltip to see the message. Moving to Done.
              chaitanya Chaitanya Kintali (Inactive) made changes -
              Assignee Chaitanya Kintali [ chaitanya ] Philip Badzuh [ pbadzuh ]
              Hide
              Permalink
              ann.loraine Ann Loraine added a comment -

              If finished with testappstore, please "stop" the EC2. Your IAM user in LoraineLab AWS account should now be able to do that. Please let me know if there are any problems.
              attn: Chaitanya Kintali

              Show
              ann.loraine Ann Loraine added a comment - If finished with testappstore, please "stop" the EC2. Your IAM user in LoraineLab AWS account should now be able to do that. Please let me know if there are any problems. attn: Chaitanya Kintali
              Hide
              Permalink
              chaitanya Chaitanya Kintali (Inactive) added a comment -

              It seems there is an internal issue that is not letting me to stop the instance.

              Show
              chaitanya Chaitanya Kintali (Inactive) added a comment - It seems there is an internal issue that is not letting me to stop the instance.
              chaitanya Chaitanya Kintali (Inactive) made changes -
              Attachment ss.PNG [ 14788 ]
              ann.loraine Ann Loraine made changes -
              Sprint Summer 3: 6 Jul - 17 Jul, Summer 4: 14 Jul - 28 Jul [ 98, 99 ] Summer 3: 6 Jul - 17 Jul, Summer 4: 14 Jul - 28 Jul, Summer 5: 29 Jul - 12 Aug [ 98, 99, 100 ]
              ann.loraine Ann Loraine made changes -
              Rank Ranked higher
              pbadzuh Philip Badzuh (Inactive) made changes -
              Resolution Done [ 10000 ]
              Status Post-merge Testing In Progress [ 10003 ] Closed [ 6 ]

                People

                • Assignee:
                  pbadzuh Philip Badzuh (Inactive)
                  Reporter:
                  ann.loraine Ann Loraine
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  3 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: