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

Try to remove the redundant code from the Help sub-app.

    Details

    • Type: Task
    • Status: Closed (View Workflow)
    • Priority: Minor
    • Resolution: Done
    • Affects Version/s: None
    • Fix Version/s: None
    • Labels:
      None
    • Story Points:
      0.5
    • Sprint:
      Spring 1 2021 Mar 22 - Apr 2, Spring 2 2021 Apr 5 - Apr 16, Spring 3 2021 Apr 19 - Apr 30

      Description

      In App Store 1.0 we were using the help sub-app in the footer but App Store 2.0 has no footer. Remove the code that are not used anywhere.

        Attachments

          Issue Links

            Activity

            chirag24 Chirag Chandrahas Shetty created issue -
            chirag24 Chirag Chandrahas Shetty made changes -
            Field Original Value New Value
            Epic Link IGBF-2563 [ 18746 ]
            chirag24 Chirag Chandrahas Shetty made changes -
            Link This issue relates to IGBF-2815 [ IGBF-2815 ]
            chirag24 Chirag Chandrahas Shetty made changes -
            Status To-Do [ 10305 ] In Progress [ 3 ]
            chirag24 Chirag Chandrahas Shetty made changes -
            Assignee Chirag Chandrahas Shetty [ chirag24 ]
            Hide
            chirag24 Chirag Chandrahas Shetty added a comment -

            Ann Loraine We cannot completely remove the help sub app because we are using one function compile-app-with-pipeline in submit-app sub app. The function simply redirects to https://docs.google.com/document/d/1JdS9vnaXHkSJb0XdR7JgExCI2Y1MhpXhN8X8okO00-0. What we can do is move this one function from help sub app to submit_app sub app or we can remove other codes from help sub-app.

            Show
            chirag24 Chirag Chandrahas Shetty added a comment - Ann Loraine We cannot completely remove the help sub app because we are using one function compile-app-with-pipeline in submit-app sub app. The function simply redirects to https://docs.google.com/document/d/1JdS9vnaXHkSJb0XdR7JgExCI2Y1MhpXhN8X8okO00-0 . What we can do is move this one function from help sub app to submit_app sub app or we can remove other codes from help sub-app.
            Hide
            aloraine Ann Loraine added a comment -

            OK.

            I think you should leave the "help" app in place but delete all functions not being used.

            Question: How are you making sure that a function is not being used?

            Show
            aloraine Ann Loraine added a comment - OK. I think you should leave the "help" app in place but delete all functions not being used. Question: How are you making sure that a function is not being used?
            Hide
            chirag24 Chirag Chandrahas Shetty added a comment -

            Ann Loraine I am searching the function name in the code base.

            Show
            chirag24 Chirag Chandrahas Shetty added a comment - Ann Loraine I am searching the function name in the code base.
            Hide
            aloraine Ann Loraine added a comment -

            Are you sure you are finding everything?

            Show
            aloraine Ann Loraine added a comment - Are you sure you are finding everything?
            Hide
            chirag24 Chirag Chandrahas Shetty added a comment -

            Yes, Dr. Ann Loraine. I will double check before removing anything and will test app-store on my local system before pushing the changes.

            Show
            chirag24 Chirag Chandrahas Shetty added a comment - Yes, Dr. Ann Loraine . I will double check before removing anything and will test app-store on my local system before pushing the changes.
            Hide
            aloraine Ann Loraine added a comment -

            Here's what I do when I need to global search:

            • move to Unix system
            • execute:
            query_string="a_method"
            find . -type f -exec grep -Iq . {} \; -print | xargs grep $query_string
            
            Show
            aloraine Ann Loraine added a comment - Here's what I do when I need to global search: move to Unix system execute: query_string= "a_method" find . -type f -exec grep -Iq . {} \; -print | xargs grep $query_string
            Hide
            chirag24 Chirag Chandrahas Shetty added a comment -

            Atom IDE as an inbuilt feature which searches the whole project for string.

            Show
            chirag24 Chirag Chandrahas Shetty added a comment - Atom IDE as an inbuilt feature which searches the whole project for string.
            chirag24 Chirag Chandrahas Shetty made changes -
            Status In Progress [ 3 ] Needs 1st Level Review [ 10005 ]
            chirag24 Chirag Chandrahas Shetty made changes -
            Assignee Chirag Chandrahas Shetty [ chirag24 ]
            Hide
            chirag24 Chirag Chandrahas Shetty added a comment - - edited

            Diff: https://bitbucket.org/chirag06/appstore/commits/7216eb1c24e59873b92fc7689a95b2a18761277d

            Testing:
            1) Please test the whole app-store by following steps mentioned in ticket https://jira.transvar.org/browse/IGBF-2717.
            2) Also test search functionality.

            Show
            chirag24 Chirag Chandrahas Shetty added a comment - - edited Diff: https://bitbucket.org/chirag06/appstore/commits/7216eb1c24e59873b92fc7689a95b2a18761277d Testing: 1) Please test the whole app-store by following steps mentioned in ticket https://jira.transvar.org/browse/IGBF-2717 . 2) Also test search functionality.
            Hide
            aloraine Ann Loraine added a comment -

            Request for Chirag Chandrahas Shetty:
            Kindly deploy the branch on one of the devappstore sites for testing and make a note of it here.
            To confirm the App store is still working properly prior to merging the PR, please go through the steps mentioned in the Jira ticket on that site, making a note of what you have done.
            In addition, click on all the "help" links (icons that look like an "i").

            Show
            aloraine Ann Loraine added a comment - Request for Chirag Chandrahas Shetty : Kindly deploy the branch on one of the devappstore sites for testing and make a note of it here. To confirm the App store is still working properly prior to merging the PR, please go through the steps mentioned in the Jira ticket on that site, making a note of what you have done. In addition, click on all the "help" links (icons that look like an "i").
            aloraine Ann Loraine made changes -
            Status Needs 1st Level Review [ 10005 ] First Level Review in Progress [ 10301 ]
            aloraine Ann Loraine made changes -
            Status First Level Review in Progress [ 10301 ] To-Do [ 10305 ]
            aloraine Ann Loraine made changes -
            Assignee Chirag Chandrahas Shetty [ chirag24 ]
            chirag24 Chirag Chandrahas Shetty made changes -
            Status To-Do [ 10305 ] In Progress [ 3 ]
            chirag24 Chirag Chandrahas Shetty made changes -
            Status In Progress [ 3 ] Needs 1st Level Review [ 10005 ]
            chirag24 Chirag Chandrahas Shetty made changes -
            Assignee Chirag Chandrahas Shetty [ chirag24 ]
            Hide
            chirag24 Chirag Chandrahas Shetty added a comment -

            Changes deployed on Devappstore5. Tested the application on devappstore 5 and everything looks good. Moving the ticket to testing.

            Show
            chirag24 Chirag Chandrahas Shetty added a comment - Changes deployed on Devappstore5. Tested the application on devappstore 5 and everything looks good. Moving the ticket to testing.
            aloraine Ann Loraine made changes -
            Status Needs 1st Level Review [ 10005 ] First Level Review in Progress [ 10301 ]
            aloraine Ann Loraine made changes -
            Assignee Ann Loraine [ aloraine ]
            aloraine Ann Loraine made changes -
            Status First Level Review in Progress [ 10301 ] Needs 1st Level Review [ 10005 ]
            aloraine Ann Loraine made changes -
            Assignee Ann Loraine [ aloraine ]
            aloraine Ann Loraine made changes -
            Status Needs 1st Level Review [ 10005 ] First Level Review in Progress [ 10301 ]
            aloraine Ann Loraine made changes -
            Assignee Ann Loraine [ aloraine ]
            aloraine Ann Loraine made changes -
            Status First Level Review in Progress [ 10301 ] Ready for Pull Request [ 10304 ]
            Hide
            aloraine Ann Loraine added a comment -

            Please submit PR when convenient.

            Show
            aloraine Ann Loraine added a comment - Please submit PR when convenient.
            aloraine Ann Loraine made changes -
            Status Ready for Pull Request [ 10304 ] Pull Request Submitted [ 10101 ]
            aloraine Ann Loraine made changes -
            Status Pull Request Submitted [ 10101 ] Reviewing Pull Request [ 10303 ]
            aloraine Ann Loraine made changes -
            Status Reviewing Pull Request [ 10303 ] To-Do [ 10305 ]
            aloraine Ann Loraine made changes -
            Status To-Do [ 10305 ] In Progress [ 3 ]
            aloraine Ann Loraine made changes -
            Status In Progress [ 3 ] Needs 1st Level Review [ 10005 ]
            aloraine Ann Loraine made changes -
            Status Needs 1st Level Review [ 10005 ] First Level Review in Progress [ 10301 ]
            aloraine Ann Loraine made changes -
            Status First Level Review in Progress [ 10301 ] Ready for Pull Request [ 10304 ]
            aloraine Ann Loraine made changes -
            Assignee Ann Loraine [ aloraine ] Chirag Chandrahas Shetty [ chirag24 ]
            aloraine Ann Loraine made changes -
            Sprint Spring 1 2021 Mar 22 - Apr 2 [ 117 ] Spring 1 2021 Mar 22 - Apr 2, Spring 2 2021 Apr 5 - Apr 16 [ 117, 118 ]
            aloraine Ann Loraine made changes -
            Rank Ranked higher
            chirag24 Chirag Chandrahas Shetty made changes -
            Status Ready for Pull Request [ 10304 ] Pull Request Submitted [ 10101 ]
            aloraine Ann Loraine made changes -
            Status Pull Request Submitted [ 10101 ] Reviewing Pull Request [ 10303 ]
            aloraine Ann Loraine made changes -
            Status Reviewing Pull Request [ 10303 ] Merged Needs Testing [ 10002 ]
            aloraine Ann Loraine made changes -
            Assignee Chirag Chandrahas Shetty [ chirag24 ]
            Show
            chirag24 Chirag Chandrahas Shetty added a comment - PR: https://bitbucket.org/lorainelab/appstore/pull-requests/276/igbf-2824-remove-code-not-used-from-help
            Hide
            aloraine Ann Loraine added a comment -
            Show
            aloraine Ann Loraine added a comment - Test on https://devappstore5.bioviz.org
            omarne Omkar Marne made changes -
            Assignee Omkar Marne [ omarne ]
            omarne Omkar Marne made changes -
            Status Merged Needs Testing [ 10002 ] Post-merge Testing In Progress [ 10003 ]
            omarne Omkar Marne made changes -
            Status Post-merge Testing In Progress [ 10003 ] Merged Needs Testing [ 10002 ]
            aloraine Ann Loraine made changes -
            Sprint Spring 1 2021 Mar 22 - Apr 2, Spring 2 2021 Apr 5 - Apr 16 [ 117, 118 ] Spring 1 2021 Mar 22 - Apr 2, Spring 2 2021 Apr 5 - Apr 16, Spring 3 2021 Apr 19 - Apr 30 [ 117, 118, 119 ]
            aloraine Ann Loraine made changes -
            Rank Ranked higher
            Hide
            omarne Omkar Marne added a comment - - edited

            Tested on https://devappstore5.bioviz.org

            • App is working fine.
            • Search functionality is working good.
            • Curated categories can be added and deleted
            • Showing the latest version when selected.
            • Asking for version number while deletion.
            Show
            omarne Omkar Marne added a comment - - edited Tested on https://devappstore5.bioviz.org App is working fine. Search functionality is working good. Curated categories can be added and deleted Showing the latest version when selected. Asking for version number while deletion.
            omarne Omkar Marne made changes -
            Status Merged Needs Testing [ 10002 ] Post-merge Testing In Progress [ 10003 ]
            omarne Omkar Marne made changes -
            Assignee Omkar Marne [ omarne ]
            aloraine Ann Loraine made changes -
            Resolution Done [ 10000 ]
            Status Post-merge Testing In Progress [ 10003 ] Closed [ 6 ]
            aloraine Ann Loraine made changes -
            Assignee Chirag Chandrahas Shetty [ chirag24 ]

              People

              • Assignee:
                chirag24 Chirag Chandrahas Shetty
                Reporter:
                chirag24 Chirag Chandrahas Shetty
              • Votes:
                0 Vote for this issue
                Watchers:
                3 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: