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
        • Epic Link:
        • 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

              Ascending order - Click to sort in descending order
              Hide
              Permalink
              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
              Hide
              Permalink
              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
              Hide
              Permalink
              pkulzer Paige Kulzer added a comment -

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

              Show
              pkulzer Paige Kulzer added a comment - PR: https://bitbucket.org/lorainelab/appstore/pull-requests/288
              Hide
              Permalink
              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."
              Hide
              Permalink
              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.
              Hide
              Permalink
              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.
              Hide
              Permalink
              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!
              Hide
              Permalink
              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.

                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: