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

Remove un-needed code - API and App dependencies

        Details

        • Type: Bug
        • Status: Closed (View Workflow)
        • Priority: Major
        • Resolution: Done
        • Affects Version/s: None
        • Fix Version/s: None
        • Labels:
        • Story Points:
          2
        • Epic Link:
        • Sprint:
          Fall 2019 Sprint 2, Spring 3 : 3 Feb to 14 Feb, Spring 3 : 17 Feb to 28 Feb, Spring 4 : 2 Mar to 13 Mar, Spring 5 : 16 Mar to 27 Mar

          Description

          AppStore codebase contains code that is not being used.
          Remove it.

            Attachments

              Issue Links

              blocks

              Bug - A problem which impairs or prevents the functions of the product. IGBF-1992 Stop using Bundle-Name

              • Major - Major loss of function.
              • Closed

                Activity

                Ascending order - Click to sort in descending order
                6 older comments
                Hide
                Permalink
                ann.loraine Ann Loraine added a comment -

                Please submit PR for IGBF-1994-1

                Show
                ann.loraine Ann Loraine added a comment - Please submit PR for IGBF-1994 -1
                Hide
                Permalink
                sameer Sameer Shanbhag (Inactive) added a comment -

                Pull Request Submitted :https://bitbucket.org/lorainelab/appstore/pull-requests/208/igbf-1994-remove-submit-api-related-code/diff

                Show
                sameer Sameer Shanbhag (Inactive) added a comment - Pull Request Submitted : https://bitbucket.org/lorainelab/appstore/pull-requests/208/igbf-1994-remove-submit-api-related-code/diff
                Hide
                Permalink
                ann.loraine Ann Loraine added a comment -

                Because the branch had some other edits, I copied over the relevant code from Sameer's branch and added it directly the master branch.
                See:
                https://bitbucket.org/lorainelab/appstore/commits/8f49e256a17506b8019c52648aa8f55512d67130
                The fix is now part of the master branch but is not yet deployed on dev-appstore-2.

                Show
                ann.loraine Ann Loraine added a comment - Because the branch had some other edits, I copied over the relevant code from Sameer's branch and added it directly the master branch. See: https://bitbucket.org/lorainelab/appstore/commits/8f49e256a17506b8019c52648aa8f55512d67130 The fix is now part of the master branch but is not yet deployed on dev-appstore-2.
                Hide
                Permalink
                ann.loraine Ann Loraine added a comment -

                FYI: master branch deployed on dev-appstore-2 up to and including these commits:

                9db2675 (HEAD -> master, origin/master, origin/HEAD) Merged in IGBF-2231 (pull request #210)
                f5753c0 IGBF-2231 : Remove old search logic | Xapian Search
                daadc80 IGBF-2231 : Remove unnecessary line of codes and organizing file imports
                bf6733a IGBF-2231 : Remove support for Geological Tracking

                Show
                ann.loraine Ann Loraine added a comment - FYI: master branch deployed on dev-appstore-2 up to and including these commits: 9db2675 (HEAD -> master, origin/master, origin/HEAD) Merged in IGBF-2231 (pull request #210) f5753c0 IGBF-2231 : Remove old search logic | Xapian Search daadc80 IGBF-2231 : Remove unnecessary line of codes and organizing file imports bf6733a IGBF-2231 : Remove support for Geological Tracking
                Hide
                Permalink
                noor91zahara Noor Zahara (Inactive) added a comment -

                Working as expected.

                Show
                noor91zahara Noor Zahara (Inactive) added a comment - Working as expected.

                  People

                  • Assignee:
                    sameer Sameer Shanbhag (Inactive)
                    Reporter:
                    ann.loraine Ann Loraine
                  • Votes:
                    0 Vote for this issue
                    Watchers:
                    4 Start watching this issue

                    Dates

                    • Created:
                      Updated:
                      Resolved: