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

Update description of the SNP File Converter in IGB App Manager

    Details

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

      Description

      Situation: The description of the SNP File Converter in the IGB App Manager currently refers to the app by its old name, "23andMe SNP Converter". Also, the app appears by its old name in the Tools menu after being installed in IGB.

      Task: For continuity, the app's description needs to be updated in the IGB App Manager. Its name will also need to be updated in the Tools menu.

        Attachments

          Issue Links

            Activity

            Hide
            pkulzer Paige Kulzer added a comment -

            I've updated the SNP File Converter's description in the IGB App Manager to remove all mention of its old name. I then also incremented the version number from 10.0.0 to 10.0.1 to ensure this new version of the description is loaded as default. Finally, I updated the app's name in the Tools menu.

            Branch: https://bitbucket.org/pkulzer-lorainelab/snp-file-converter/branch/IGBF-3852?dest=lorainelab%2Fsnp-file-converter%3Amaster

            Show
            pkulzer Paige Kulzer added a comment - I've updated the SNP File Converter's description in the IGB App Manager to remove all mention of its old name. I then also incremented the version number from 10.0.0 to 10.0.1 to ensure this new version of the description is loaded as default. Finally, I updated the app's name in the Tools menu. Branch: https://bitbucket.org/pkulzer-lorainelab/snp-file-converter/branch/IGBF-3852?dest=lorainelab%2Fsnp-file-converter%3Amaster
            Hide
            nfreese Nowlan Freese added a comment -

            Two suggested changes:

            • In SnpConverterFrame.java there is a line setTitle("23&Me Converter"); that sets the title of the window that opens when the user starts the app. Should this be changed to something like "SNP Converter"?
            • I think it would be good to pull the change to the .gitignore into its own commit, as it is not directly related to the changes to the updated description of the SNP converter. This stackoverflow talks about how to make the change.
            Show
            nfreese Nowlan Freese added a comment - Two suggested changes: In SnpConverterFrame.java there is a line setTitle("23&Me Converter"); that sets the title of the window that opens when the user starts the app. Should this be changed to something like "SNP Converter"? I think it would be good to pull the change to the .gitignore into its own commit, as it is not directly related to the changes to the updated description of the SNP converter. This stackoverflow talks about how to make the change.
            Hide
            pkulzer Paige Kulzer added a comment -

            Good catch!

            I've updated that line in SnpConverterFrame.java to set the title as "SNP File Converter" instead. I also broke up my commits like you suggested, so you should now see two commits – one for the change to my .gitignore file and another with all of the app-related changes.

            Show
            pkulzer Paige Kulzer added a comment - Good catch! I've updated that line in SnpConverterFrame.java to set the title as "SNP File Converter" instead. I also broke up my commits like you suggested, so you should now see two commits – one for the change to my .gitignore file and another with all of the app-related changes.
            Hide
            nfreese Nowlan Freese added a comment -

            Changes and commit look good.

            Ready for pull request.

            Show
            nfreese Nowlan Freese added a comment - Changes and commit look good. Ready for pull request.
            Show
            pkulzer Paige Kulzer added a comment - PR: https://bitbucket.org/lorainelab/snp-file-converter/pull-requests/14
            Hide
            ann.loraine Ann Loraine added a comment - - edited

            PR is now merged, the build pipeline on bitbucket has completed, and there is a new jar file available for testing in the Downloads section of the main repository.

            Ready for testing.

            Show
            ann.loraine Ann Loraine added a comment - - edited PR is now merged, the build pipeline on bitbucket has completed, and there is a new jar file available for testing in the Downloads section of the main repository. Ready for testing.
            Hide
            nfreese Nowlan Freese added a comment -

            Tested using the lorainelab Downloads section of the SNP file converter repository on Mac.
            Description looks good, app installed correctly, app title was updated.

            Closing ticket.

            Show
            nfreese Nowlan Freese added a comment - Tested using the lorainelab Downloads section of the SNP file converter repository on Mac. Description looks good, app installed correctly, app title was updated. Closing ticket.
            Hide
            pkulzer Paige Kulzer added a comment - - edited

            I'm re-opening this ticket because we missed a line of code in the pom.xml which still referred to the app by its old name.

            Branch: https://bitbucket.org/pkulzer-lorainelab/snp-file-converter/branch/IGBF-3852?dest=lorainelab%2Fsnp-file-converter%3Amaster

            EDIT: This line of code (i.e., the artifact ID) should not be changed in the future. This ID is what the IGB App Store uses to recognize apps, so when I went to update the SNP File Converter to a new version in the App Store with this artifact ID changed, it detected the SNP-File-Converter-10.0.1.jar file as a completely new app.

            Closing this ticket once again!

            Show
            pkulzer Paige Kulzer added a comment - - edited I'm re-opening this ticket because we missed a line of code in the pom.xml which still referred to the app by its old name. Branch: https://bitbucket.org/pkulzer-lorainelab/snp-file-converter/branch/IGBF-3852?dest=lorainelab%2Fsnp-file-converter%3Amaster EDIT: This line of code (i.e., the artifact ID) should not be changed in the future. This ID is what the IGB App Store uses to recognize apps, so when I went to update the SNP File Converter to a new version in the App Store with this artifact ID changed, it detected the SNP-File-Converter-10.0.1.jar file as a completely new app. Closing this ticket once again!

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved: