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

Fix contig name error in ProtAnnot window title

        Details

        • Story Points:
          0.5
        • Sprint:
          Summer 2018 Part 2, Summer 2018 Part 3, Fall 2018 1, Fall 2018 Sprint 2

          Description

          The ProtAnnot header bar gives chrChr1 as the contig. It should be either chr1 or Chr1.

            Attachments

              Issue Links

              relates to

              Bug - A problem which impairs or prevents the functions of the product. IGBF-1108 App Manager does not properly handle installation error when not connected to the internet

              • Major - Major loss of function.
              • Closed

                Activity

                Ascending order - Click to sort in descending order
                4 older comments
                Hide
                Permalink
                ieclabau Ivory Blakley (Inactive) added a comment -

                code review,
                Looking at the changes in bitbucket, the change is very minor, and I do not see any potential for side effects.

                git review
                All changes are in a single commit with an appropriate message.
                The branch is currently two commits behind Anh's master, so it needs to be rebased.

                I am reassigning this to Anh to do the rebase and then submit a pull request.
                Anh, be sure to update your local master from upstream master, then rebase this branch onto your master, and push both master and this branch to your remote. Check that the branch says 'ahead by 1 commit' relative to your master.

                Show
                ieclabau Ivory Blakley (Inactive) added a comment - code review, Looking at the changes in bitbucket, the change is very minor, and I do not see any potential for side effects. git review All changes are in a single commit with an appropriate message. The branch is currently two commits behind Anh's master, so it needs to be rebased. I am reassigning this to Anh to do the rebase and then submit a pull request. Anh, be sure to update your local master from upstream master, then rebase this branch onto your master, and push both master and this branch to your remote. Check that the branch says 'ahead by 1 commit' relative to your master.
                Hide
                Permalink
                ann.loraine Ann Loraine added a comment -

                Pull request merged.

                Show
                ann.loraine Ann Loraine added a comment - Pull request merged.
                Hide
                Permalink
                mason Mason Meyer (Inactive) added a comment -

                I am not currently seeing this issue as resolved. When I open ProtAnnot using the IGB App Manager, I am seeing content in the title bar such as "chrChr1". Should I be testing this story a different way besides using the App Manager? For now, I am re-assigning this story to Ann and moving to To-Do.

                Show
                mason Mason Meyer (Inactive) added a comment - I am not currently seeing this issue as resolved. When I open ProtAnnot using the IGB App Manager, I am seeing content in the title bar such as "chrChr1". Should I be testing this story a different way besides using the App Manager? For now, I am re-assigning this story to Ann and moving to To-Do.
                Hide
                Permalink
                ieclabau Ivory Blakley (Inactive) added a comment -

                After clarifying the testing protocol, I am moving this back to Needs Testing and assigning it to Mason.

                Show
                ieclabau Ivory Blakley (Inactive) added a comment - After clarifying the testing protocol, I am moving this back to Needs Testing and assigning it to Mason.
                Hide
                Permalink
                mason Mason Meyer (Inactive) added a comment -

                My testing confirms that the contig name error is fixed in ProtAnnot. I am no longer seeing the additional "chr" text in the window title. I am also not experiencing any side effects related to this change, so the story will now be closed.

                Show
                mason Mason Meyer (Inactive) added a comment - My testing confirms that the contig name error is fixed in ProtAnnot. I am no longer seeing the additional "chr" text in the window title. I am also not experiencing any side effects related to this change, so the story will now be closed.

                  People

                  • Assignee:
                    aduong Anh Moss (Inactive)
                    Reporter:
                    ieclabau Ivory Blakley (Inactive)
                  • Votes:
                    0 Vote for this issue
                    Watchers:
                    Start watching this issue

                    Dates

                    • Created:
                      Updated:
                      Resolved: