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

Display accepted analysis filetypes

    Details

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

      Description

      Situation: When a user right clicks and selects Analyse for a filetype that does not have any associated App, the user is only shown the file name in the Analysis right panel. This may be confusing to users.

      Task: If no App is available for analysing the selected file type, show the user text in the Analysis right panel that tells the user what filetypes can be analysed. The accepted Analysis file types are kept in the database and are updated when we run the sync apps database endpoint.

        Attachments

        1. current.png
          current.png
          13 kB
        2. NoAppsAvailable.png
          NoAppsAvailable.png
          114 kB
        3. proposed.png
          proposed.png
          24 kB

          Issue Links

            Activity

            nfreese Nowlan Freese created issue -
            nfreese Nowlan Freese made changes -
            Field Original Value New Value
            Epic Link IGBF-2376 [ 18533 ]
            nfreese Nowlan Freese made changes -
            Link This issue relates to IGBF-3218 [ IGBF-3218 ]
            nfreese Nowlan Freese made changes -
            Description Situation: When a user right clicks and selects Analyse for a filetype that does not have any associated App, the user is only shown the file name in the Analysis right panel. This may be confusing to users.

            Task: If no App is available for analysing the selected file type, show the user text in the Analysis right panel that tells the user what filetypes can be analysed.
            Situation: When a user right clicks and selects Analyse for a filetype that does not have any associated App, the user is only shown the file name in the Analysis right panel. This may be confusing to users.

            Task: If no App is available for analysing the selected file type, show the user text in the Analysis right panel that tells the user what filetypes can be analysed. The accepted Analysis file types are kept in the database and are updated when we run the sync apps database endpoint.
            nfreese Nowlan Freese made changes -
            Attachment current.png [ 17554 ]
            Attachment proposed.png [ 17555 ]
            Hide
            nfreese Nowlan Freese added a comment -

            See attached images for current Analysis right panel and proposed Analysis right panel when no app is available for the selected file type.

            Show
            nfreese Nowlan Freese added a comment - See attached images for current Analysis right panel and proposed Analysis right panel when no app is available for the selected file type.
            nfreese Nowlan Freese made changes -
            Status To-Do [ 10305 ] In Progress [ 3 ]
            Hide
            karthik Karthik Raveendran added a comment -

            Changes are pushed. See commit

            Show
            karthik Karthik Raveendran added a comment - Changes are pushed. See commit
            karthik Karthik Raveendran made changes -
            Status In Progress [ 3 ] Needs 1st Level Review [ 10005 ]
            karthik Karthik Raveendran made changes -
            Assignee Karthik Raveendran [ karthik ]
            nfreese Nowlan Freese made changes -
            Status Needs 1st Level Review [ 10005 ] First Level Review in Progress [ 10301 ]
            nfreese Nowlan Freese made changes -
            Status First Level Review in Progress [ 10301 ] Ready for Pull Request [ 10304 ]
            Hide
            nfreese Nowlan Freese added a comment -

            Looks good, please create a pull request.

            Show
            nfreese Nowlan Freese added a comment - Looks good, please create a pull request.
            karthik Karthik Raveendran made changes -
            Status Ready for Pull Request [ 10304 ] Pull Request Submitted [ 10101 ]
            Show
            karthik Karthik Raveendran added a comment - PR Submitted. https://bitbucket.org/nfreese/bioviz-connect/pull-requests/288/igbf-3224-display-accepted-analysis
            Hide
            nfreese Nowlan Freese added a comment -

            Merged

            Show
            nfreese Nowlan Freese added a comment - Merged
            nfreese Nowlan Freese made changes -
            Status Pull Request Submitted [ 10101 ] Reviewing Pull Request [ 10303 ]
            nfreese Nowlan Freese made changes -
            Status Reviewing Pull Request [ 10303 ] Merged Needs Testing [ 10002 ]
            Hide
            ann.loraine Ann Loraine added a comment -

            Updated on bvctest3.bioviz.org.

            Show
            ann.loraine Ann Loraine added a comment - Updated on bvctest3.bioviz.org.
            nfreese Nowlan Freese made changes -
            Status Merged Needs Testing [ 10002 ] Post-merge Testing In Progress [ 10003 ]
            nfreese Nowlan Freese made changes -
            Assignee Nowlan Freese [ nfreese ]
            Hide
            nfreese Nowlan Freese added a comment -

            Working correctly.

            Closing ticket.

            Show
            nfreese Nowlan Freese added a comment - Working correctly. Closing ticket.
            nfreese Nowlan Freese made changes -
            Status Post-merge Testing In Progress [ 10003 ] Merged Needs Testing [ 10002 ]
            nfreese Nowlan Freese made changes -
            Status Merged Needs Testing [ 10002 ] Post-merge Testing In Progress [ 10003 ]
            nfreese Nowlan Freese made changes -
            Resolution Done [ 10000 ]
            Status Post-merge Testing In Progress [ 10003 ] Closed [ 6 ]
            nfreese Nowlan Freese made changes -
            Assignee Nowlan Freese [ nfreese ] Karthik Raveendran [ karthik ]
            rweidenh Logan Weidenhammer (Inactive) made changes -
            Attachment NoAppsAvailable.png [ 17565 ]
            Hide
            rweidenh Logan Weidenhammer (Inactive) added a comment -

            BioViz Connect is showing

            "No Apps are available for analyzing this filetype."

            even when there are analyses available for that file.

            Show
            rweidenh Logan Weidenhammer (Inactive) added a comment - BioViz Connect is showing "No Apps are available for analyzing this filetype." even when there are analyses available for that file.
            rweidenh Logan Weidenhammer (Inactive) made changes -
            Resolution Done [ 10000 ]
            Status Closed [ 6 ] To-Do [ 10305 ]
            Hide
            nfreese Nowlan Freese added a comment -

            We are seeing a number of issues with BioViz Connect, pages not loading, login taking multiple attempts. It is unclear if this is a CyVerse issue or a BioViz Connect server issue. I'm somewhat suspect due to the issues with running the playbooks today, but BioViz Connect production and bvctest3 are behaving similarly (in terms of quirks and odd behavior) so maybe it is a CyVerse issue? But the Terrain endpoints seem to be behaving normally when I hit them through Postman. For this issue specifically, the right-panel is showing correctly for me. However, this is odd in itself as the analysis right panel is constructed from our database and not through the Terrain endpoints.

            Show
            nfreese Nowlan Freese added a comment - We are seeing a number of issues with BioViz Connect, pages not loading, login taking multiple attempts. It is unclear if this is a CyVerse issue or a BioViz Connect server issue. I'm somewhat suspect due to the issues with running the playbooks today, but BioViz Connect production and bvctest3 are behaving similarly (in terms of quirks and odd behavior) so maybe it is a CyVerse issue? But the Terrain endpoints seem to be behaving normally when I hit them through Postman. For this issue specifically, the right-panel is showing correctly for me. However, this is odd in itself as the analysis right panel is constructed from our database and not through the Terrain endpoints.
            nfreese Nowlan Freese made changes -
            Assignee Karthik Raveendran [ karthik ]
            Hide
            ann.loraine Ann Loraine added a comment - - edited

            Deployed code to a new host: 34.202.1.73
            Django code given domain name: connect2.bioviz.org

            Problems observed:

            • sync'ing script "sync_app_data.sh" fails

            Possible problem observed:

            • make migrations step failing to create a new migrations script, not sure if this matters or not

            Please note:

            • KR, NF, AL public keys installed on the host to enable them to log in to the host
            Show
            ann.loraine Ann Loraine added a comment - - edited Deployed code to a new host: 34.202.1.73 Django code given domain name: connect2.bioviz.org Problems observed: sync'ing script "sync_app_data.sh" fails Possible problem observed: make migrations step failing to create a new migrations script, not sure if this matters or not Please note: KR, NF, AL public keys installed on the host to enable them to log in to the host
            Hide
            ann.loraine Ann Loraine added a comment -

            Problems with the API are complicating further testing of this issue.
            Yesterday NF tested the code change on a test host and observed expected behavior.
            Closing for now as the current problems with BioViz Connect are judged to be not due to these code changes.

            Show
            ann.loraine Ann Loraine added a comment - Problems with the API are complicating further testing of this issue. Yesterday NF tested the code change on a test host and observed expected behavior. Closing for now as the current problems with BioViz Connect are judged to be not due to these code changes.
            ann.loraine Ann Loraine made changes -
            Status To-Do [ 10305 ] In Progress [ 3 ]
            ann.loraine Ann Loraine made changes -
            Status In Progress [ 3 ] Needs 1st Level Review [ 10005 ]
            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 -
            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 ]
            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 Karthik Raveendran [ karthik ]

              People

              • Assignee:
                karthik Karthik Raveendran
                Reporter:
                nfreese Nowlan Freese
              • Votes:
                0 Vote for this issue
                Watchers:
                4 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: