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

Alter text on IGB App Store regarding which version of IGB to run Apps

    Details

    • Type: Task
    • Status: Closed (View Workflow)
    • Priority: Major
    • Resolution: Done
    • Affects Version/s: None
    • Fix Version/s: None
    • Labels:
      None
    • Story Points:
      1
    • Sprint:
      Fall 1, Summer 6, Fall 2, Fall 3, Fall 4, Fall 5, Fall 6, Fall 7, Fall 8

      Description

      Situation: When a user views the IGB App Store (for example https://apps.bioviz.org/apps/downsampler-filter) without IGB running they see a message that says "To install an App, start IGB version 9.1.0 or later. Then reload this page." The 9.1.0 comes from changes to the IGB codebase that were required to communicate with the App Store. However, several new apps only work with IGB versions over 10.0.0. This may confuse users as two different IGB versions are being recommended.

      Task: Update the text to say something such as:

      • "To install an App, start the latest version of IGB. Then reload this page."
      • "To install an App, start IGB. Then reload this page."

        Attachments

          Activity

          nfreese Nowlan Freese created issue -
          nfreese Nowlan Freese made changes -
          Field Original Value New Value
          Epic Link IGBF-1908 [ 17998 ]
          ann.loraine Ann Loraine made changes -
          Sprint Summer 6 [ 223 ] Testing 3 : 19 Nov - 29 Nov 2, Summer 6 [ 83, 223 ]
          ann.loraine Ann Loraine made changes -
          Rank Ranked higher
          ann.loraine Ann Loraine made changes -
          Sprint Fall 1, Summer 6 [ 83, 223 ] Fall 1, Summer 6, Fall 2 [ 83, 223, 225 ]
          ann.loraine Ann Loraine made changes -
          Rank Ranked higher
          ann.loraine Ann Loraine made changes -
          Sprint Fall 1, Summer 6, Fall 2 [ 83, 223, 225 ] Fall 1, Summer 6, Fall 2, Fall 3 [ 83, 223, 225, 226 ]
          ann.loraine Ann Loraine made changes -
          Rank Ranked higher
          ann.loraine Ann Loraine made changes -
          Sprint Fall 1, Summer 6, Fall 2, Fall 3 [ 83, 223, 225, 226 ] Fall 1, Summer 6, Fall 2, Fall 3, Fall 4 [ 83, 223, 225, 226, 227 ]
          ann.loraine Ann Loraine made changes -
          Rank Ranked higher
          ann.loraine Ann Loraine made changes -
          Sprint Fall 1, Summer 6, Fall 2, Fall 3, Fall 4 [ 83, 223, 225, 226, 227 ] Fall 1, Summer 6, Fall 2, Fall 3, Fall 4, Fall 5 [ 83, 223, 225, 226, 227, 228 ]
          ann.loraine Ann Loraine made changes -
          Rank Ranked higher
          nfreese Nowlan Freese made changes -
          Sprint Fall 1, Summer 6, Fall 2, Fall 3, Fall 4, Fall 5 [ 83, 223, 225, 226, 227, 228 ] Fall 1, Summer 6, Fall 2, Fall 3, Fall 4, Fall 5, Fall 6 [ 83, 223, 225, 226, 227, 228, 229 ]
          nfreese Nowlan Freese made changes -
          Rank Ranked higher
          ann.loraine Ann Loraine made changes -
          Sprint Fall 1, Summer 6, Fall 2, Fall 3, Fall 4, Fall 5, Fall 6 [ 83, 223, 225, 226, 227, 228, 229 ] Fall 1, Summer 6, Fall 2, Fall 3, Fall 4, Fall 5, Fall 6, Fall 7 [ 83, 223, 225, 226, 227, 228, 229, 230 ]
          ann.loraine Ann Loraine made changes -
          Rank Ranked higher
          pkulzer Paige Kulzer made changes -
          Assignee Paige Kulzer [ pkulzer ]
          pkulzer Paige Kulzer made changes -
          Status To-Do [ 10305 ] In Progress [ 3 ]
          Hide
          pkulzer Paige Kulzer added a comment -

          I found the instances of that message in the App Store codebase and modified them to the suggested text. Ready for review!

          Branch: https://bitbucket.org/pkulzer-lorainelab/appstore/branch/IGBF-4284

          Show
          pkulzer Paige Kulzer added a comment - I found the instances of that message in the App Store codebase and modified them to the suggested text. Ready for review! Branch: https://bitbucket.org/pkulzer-lorainelab/appstore/branch/IGBF-4284
          pkulzer Paige Kulzer made changes -
          Status In Progress [ 3 ] Needs 1st Level Review [ 10005 ]
          pkulzer Paige Kulzer made changes -
          Assignee Paige Kulzer [ pkulzer ] Nowlan Freese [ nfreese ]
          ann.loraine Ann Loraine made changes -
          Sprint Fall 1, Summer 6, Fall 2, Fall 3, Fall 4, Fall 5, Fall 6, Fall 7 [ 83, 223, 225, 226, 227, 228, 229, 230 ] Fall 1, Summer 6, Fall 2, Fall 3, Fall 4, Fall 5, Fall 6, Fall 7, Fall 8 [ 83, 223, 225, 226, 227, 228, 229, 230, 231 ]
          ann.loraine Ann Loraine made changes -
          Rank Ranked higher
          nfreese Nowlan Freese made changes -
          Status Needs 1st Level Review [ 10005 ] First Level Review in Progress [ 10301 ]
          Hide
          nfreese Nowlan Freese added a comment -

          Tested on Mac

          Text is changed to the new message: 'To install an App, start the latest version of IGB. Then reload this page.', 'info'

          Ready for PR

          Show
          nfreese Nowlan Freese added a comment - Tested on Mac Text is changed to the new message: 'To install an App, start the latest version of IGB. Then reload this page.', 'info' Ready for PR
          nfreese Nowlan Freese made changes -
          Assignee Nowlan Freese [ nfreese ] Paige Kulzer [ pkulzer ]
          nfreese Nowlan Freese made changes -
          Status First Level Review in Progress [ 10301 ] Ready for Pull Request [ 10304 ]
          Show
          pkulzer Paige Kulzer added a comment - PR: https://bitbucket.org/lorainelab/appstore/pull-requests/288
          pkulzer Paige Kulzer made changes -
          Status Ready for Pull Request [ 10304 ] Pull Request Submitted [ 10101 ]
          pkulzer Paige Kulzer made changes -
          Assignee Paige Kulzer [ pkulzer ] Ann Loraine [ aloraine ]
          Hide
          ann.loraine Ann Loraine added a comment - - edited

          This was one of the two options suggested Nowlan Freese wrote the ticket.

          I think the shorter option is probably better.

          That's because even saying "latest version of IGB" is a little ambiguous. That could mean whatever version is available in the "early access" section of the site.

          I would like change the wording to the other option I suggested:

          The "latest version of IGB" might be confusing because a user might think we are talking about the literal latest version, which is usually the version being developed.

          Request for Paige Kulzer:

          Would you change the wording to:

          "To install an App, start IGB. Then reload this page."

          Show
          ann.loraine Ann Loraine added a comment - - edited This was one of the two options suggested Nowlan Freese wrote the ticket. I think the shorter option is probably better. That's because even saying "latest version of IGB" is a little ambiguous. That could mean whatever version is available in the "early access" section of the site. I would like change the wording to the other option I suggested: The "latest version of IGB" might be confusing because a user might think we are talking about the literal latest version, which is usually the version being developed. Request for Paige Kulzer : Would you change the wording to: "To install an App, start IGB. Then reload this page."
          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 ] To-Do [ 10305 ]
          Hide
          pkulzer Paige Kulzer added a comment -

          The PR has been updated with the changed wording.

          Show
          pkulzer Paige Kulzer added a comment - The PR has been updated with the changed wording.
          pkulzer Paige Kulzer made changes -
          Status To-Do [ 10305 ] In Progress [ 3 ]
          pkulzer Paige Kulzer made changes -
          Status In Progress [ 3 ] Needs 1st Level Review [ 10005 ]
          pkulzer Paige Kulzer made changes -
          Status Needs 1st Level Review [ 10005 ] First Level Review in Progress [ 10301 ]
          pkulzer Paige Kulzer made changes -
          Status First Level Review in Progress [ 10301 ] Ready for Pull Request [ 10304 ]
          pkulzer Paige Kulzer made changes -
          Status Ready for Pull Request [ 10304 ] Pull Request Submitted [ 10101 ]
          ann.loraine Ann Loraine made changes -
          Status Pull Request Submitted [ 10101 ] Reviewing Pull Request [ 10303 ]
          Hide
          ann.loraine Ann Loraine added a comment - - edited

          I am a little nervous about merging this because there is some logic related to "xhr.status" that seems important.

          The reason (I think) that the specific language about IGB version is that the version mentioned was the first version of IGB that could communicate with App Store, via localhost requests.

          Until this can be tested on a test server, I don't want to merge this change.

          A request for Paige Kulzer:

          Would you please submit a PR to the main repository for your branch – not a merge request? What I would like to do is have the branch become part of the team repository so that we can more easily come back to this once the appstore playbooks are working again.

          Also, notifying Nowlan Freese to keep you in the loop.

          Show
          ann.loraine Ann Loraine added a comment - - edited I am a little nervous about merging this because there is some logic related to "xhr.status" that seems important. The reason (I think) that the specific language about IGB version is that the version mentioned was the first version of IGB that could communicate with App Store, via localhost requests. Until this can be tested on a test server, I don't want to merge this change. A request for Paige Kulzer : Would you please submit a PR to the main repository for your branch – not a merge request? What I would like to do is have the branch become part of the team repository so that we can more easily come back to this once the appstore playbooks are working again. Also, notifying Nowlan Freese to keep you in the loop.
          ann.loraine Ann Loraine made changes -
          Assignee Ann Loraine [ aloraine ] Paige Kulzer [ pkulzer ]
          Hide
          pkulzer Paige Kulzer added a comment -

          PR: https://bitbucket.org/lorainelab/appstore/pull-requests/289

          Ann Loraine, I made a PR to the Loraine Lab repo to make a new branch there with my changes. Please just let me know if that's not what you were requesting!

          Show
          pkulzer Paige Kulzer added a comment - PR: https://bitbucket.org/lorainelab/appstore/pull-requests/289 Ann Loraine , I made a PR to the Loraine Lab repo to make a new branch there with my changes. Please just let me know if that's not what you were requesting!
          pkulzer Paige Kulzer made changes -
          Status Reviewing Pull Request [ 10303 ] Pull Request Submitted [ 10101 ]
          pkulzer Paige Kulzer made changes -
          Assignee Paige Kulzer [ pkulzer ] Ann Loraine [ aloraine ]
          Hide
          ann.loraine Ann Loraine added a comment -

          Thank you Paige Kulzer!

          Using the bitbucket interface I selected "merge" and this created a new branch on the team repository: https://bitbucket.org/lorainelab/appstore/branch/IGBF-4284

          Moving to DONE.

          Show
          ann.loraine Ann Loraine added a comment - Thank you Paige Kulzer ! Using the bitbucket interface I selected "merge" and this created a new branch on the team repository: https://bitbucket.org/lorainelab/appstore/branch/IGBF-4284 Moving to DONE.
          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 ]
          ann.loraine Ann Loraine made changes -
          Status Merged Needs Testing [ 10002 ] Post-merge Testing In Progress [ 10003 ]
          ann.loraine Ann Loraine made changes -
          Resolution Done [ 10000 ]
          Status Post-merge Testing In Progress [ 10003 ] Closed [ 6 ]
          ann.loraine Ann Loraine made changes -
          Assignee Ann Loraine [ aloraine ] Paige Kulzer [ pkulzer ]

            People

            • Assignee:
              pkulzer Paige Kulzer
              Reporter:
              nfreese Nowlan Freese
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: