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

Remove Variables from SubmitApp Codebase

    Details

    • Type: Task
    • Status: Closed (View Workflow)
    • Priority: Major
    • Resolution: Done
    • Affects Version/s: None
    • Fix Version/s: None
    • Labels:
      None
    • Story Points:
      1.5
    • Sprint:
      Winter 1 Dec 28 - Jan 8, Winter 2 Jan 11 - Jan 22, Winter 3 Jan 25 - Feb 5, Winter 4 Feb 8 - Feb 19

      Description

      Currently Submit App has two variables which are causing some error looking warning in the log. These variables does not break anything but just creates a lot of error messages in log which can be demeaning.

      expect_app_name and is_pending_replace needs to be taken care of

        Attachments

          Activity

          sameer Sameer Shanbhag (Inactive) created issue -
          sameer Sameer Shanbhag (Inactive) made changes -
          Field Original Value New Value
          Epic Link IGBF-2563 [ 18746 ]
          sameer Sameer Shanbhag (Inactive) made changes -
          Status To-Do [ 10305 ] Pull Request Submitted [ 10101 ]
          Hide
          sameer Sameer Shanbhag (Inactive) added a comment -

          Pull Request:

          Change Log:

          • Removed Expect App Name Variable

          P.S. This can be used as a reference if we plan on removing these variables later in the project

          C.C. [~aloraine]

          Show
          sameer Sameer Shanbhag (Inactive) added a comment - Pull Request: https://bitbucket.org/lorainelab/appstore/pull-requests/265/igbf-2747-remove-expect-app-name-from-the Change Log: Removed Expect App Name Variable P.S. This can be used as a reference if we plan on removing these variables later in the project C.C. [~aloraine]
          ssegu Sai Supreeth Segu (Inactive) made changes -
          Status Pull Request Submitted [ 10101 ] Reviewing Pull Request [ 10303 ]
          ssegu Sai Supreeth Segu (Inactive) made changes -
          Status Reviewing Pull Request [ 10303 ] Pull Request Submitted [ 10101 ]
          ann.loraine Ann Loraine made changes -
          Sprint Winter 1 Dec 28 - Jan 8 [ 111 ] Winter 1 Dec 28 - Jan 8, Winter 2 Jan 11 - Jan 22 [ 111, 112 ]
          ann.loraine Ann Loraine made changes -
          Rank Ranked higher
          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 -
          Assignee Sameer Shanbhag [ sameer ]
          Hide
          ann.loraine Ann Loraine added a comment -

          deployed to devappstore3 and devappstore5

          Show
          ann.loraine Ann Loraine added a comment - deployed to devappstore3 and devappstore5
          chirag24 Chirag Chandrahas Shetty (Inactive) made changes -
          Assignee Chirag Chandrahas Shetty [ chirag24 ]
          chirag24 Chirag Chandrahas Shetty (Inactive) made changes -
          Status Merged Needs Testing [ 10002 ] Post-merge Testing In Progress [ 10003 ]
          Hide
          chirag24 Chirag Chandrahas Shetty (Inactive) added a comment -

          Still getting error msg in the logs.

          Traceback (most recent call last):
          File "/home/ubuntu/appstore/venv/lib/python3.6/site-packages/django/template/base.py", line 843, in _resolve_lookup
          current = current[int(bit)]
          ValueError: invalid literal for int() with base 10: 'info_msg'

          During handling of the above exception, another exception occurred:

          Traceback (most recent call last):
          File "/home/ubuntu/appstore/venv/lib/python3.6/site-packages/django/template/base.py", line 850, in _resolve_lookup
          (bit, current)) # missing attribute
          django.template.base.VariableDoesNotExist: Failed lookup for key [info_msg] in [

          {'True': True, 'False': False, 'None': None}

          , {'csrf_token': <SimpleLazyObject: <function csrf.<locals>._get_val at 0x7fa948a20d90>>, 'request': <WSGIRequest: POST '/submit_app/confirm/8'>, 'user': <SimpleLazyObject: <User: cshetty>>, 'perms': <django.contrib.auth.context_processors.PermWrapper object at 0x7fa9489deef0>, 'messages': <django.contrib.messages.storage.fallback.FallbackStorage object at 0x7fa9489dee48>, 'DEFAULT_MESSAGE_LEVELS':

          {'DEBUG': 10, 'INFO': 20, 'SUCCESS': 25, 'WARNING': 30, 'ERROR': 40}

          , 'backends': <LazyDict: <function backends.<locals>.<lambda> at 0x7fa949ac3400>>, 'REDIRECT_FIELD_NAME': 'next', 'REDIRECT_FIELD_VALUE': None, 'REDIRECT_QUERYSTRING': ''}, {},

          {'app_name': 'Show Splice Support'}

          ,

          {'block': <Block Node: error_pop. Contents: [<TextNode: '\n '>, <django.template.loader_tags.IncludeNode object at 0x7fa9489e20f0>, <TextNode: '\n '>]>}

          , {},

          {'block': <Block Node: error_pop. Contents: [<TextNode: '\n<div class="col-md-12" i'>, <IfNode>, <TextNode: '\n '>, <IfNode>, <TextNode: '\n</div>\n'>]>}

          ]

          Show
          chirag24 Chirag Chandrahas Shetty (Inactive) added a comment - Still getting error msg in the logs. Traceback (most recent call last): File "/home/ubuntu/appstore/venv/lib/python3.6/site-packages/django/template/base.py", line 843, in _resolve_lookup current = current [int(bit)] ValueError: invalid literal for int() with base 10: 'info_msg' During handling of the above exception, another exception occurred: Traceback (most recent call last): File "/home/ubuntu/appstore/venv/lib/python3.6/site-packages/django/template/base.py", line 850, in _resolve_lookup (bit, current)) # missing attribute django.template.base.VariableDoesNotExist: Failed lookup for key [info_msg] in [ {'True': True, 'False': False, 'None': None} , {'csrf_token': <SimpleLazyObject: <function csrf.<locals>._get_val at 0x7fa948a20d90>>, 'request': <WSGIRequest: POST '/submit_app/confirm/8'>, 'user': <SimpleLazyObject: <User: cshetty>>, 'perms': <django.contrib.auth.context_processors.PermWrapper object at 0x7fa9489deef0>, 'messages': <django.contrib.messages.storage.fallback.FallbackStorage object at 0x7fa9489dee48>, 'DEFAULT_MESSAGE_LEVELS': {'DEBUG': 10, 'INFO': 20, 'SUCCESS': 25, 'WARNING': 30, 'ERROR': 40} , 'backends': <LazyDict: <function backends.<locals>.<lambda> at 0x7fa949ac3400>>, 'REDIRECT_FIELD_NAME': 'next', 'REDIRECT_FIELD_VALUE': None, 'REDIRECT_QUERYSTRING': ''}, {}, {'app_name': 'Show Splice Support'} , {'block': <Block Node: error_pop. Contents: [<TextNode: '\n '>, <django.template.loader_tags.IncludeNode object at 0x7fa9489e20f0>, <TextNode: '\n '>]>} , {}, {'block': <Block Node: error_pop. Contents: [<TextNode: '\n<div class="col-md-12" i'>, <IfNode>, <TextNode: '\n '>, <IfNode>, <TextNode: '\n</div>\n'>]>} ]
          chirag24 Chirag Chandrahas Shetty (Inactive) made changes -
          Status Post-merge Testing In Progress [ 10003 ] To-Do [ 10305 ]
          chirag24 Chirag Chandrahas Shetty (Inactive) made changes -
          Assignee Chirag Chandrahas Shetty [ chirag24 ]
          ann.loraine Ann Loraine made changes -
          Sprint Winter 1 Dec 28 - Jan 8, Winter 2 Jan 11 - Jan 22 [ 111, 112 ] Winter 1 Dec 28 - Jan 8, Winter 2 Jan 11 - Jan 22, Winter 3 Jan 25 - Feb 5 [ 111, 112, 113 ]
          ann.loraine Ann Loraine made changes -
          Rank Ranked higher
          Hide
          ann.loraine Ann Loraine added a comment -

          Chirag Chandrahas Shetty - the above errors do not mention the missing variables. Does this confirm that Sameer's PR has corrected the specific problem with those variables?

          Please also inspect the code change itself.

          Show
          ann.loraine Ann Loraine added a comment - Chirag Chandrahas Shetty - the above errors do not mention the missing variables. Does this confirm that Sameer's PR has corrected the specific problem with those variables? Please also inspect the code change itself.
          ann.loraine Ann Loraine made changes -
          Assignee Chirag Chandrahas Shetty [ chirag24 ]
          chirag24 Chirag Chandrahas Shetty (Inactive) made changes -
          Status To-Do [ 10305 ] In Progress [ 3 ]
          chirag24 Chirag Chandrahas Shetty (Inactive) made changes -
          Status In Progress [ 3 ] To-Do [ 10305 ]
          chirag24 Chirag Chandrahas Shetty (Inactive) made changes -
          Status To-Do [ 10305 ] In Progress [ 3 ]
          Hide
          chirag24 Chirag Chandrahas Shetty (Inactive) added a comment -

          [~aloraine] Only one variable is removed from the code base ie: Expect App Name Variable. The code looks fine. I looked into the log files(debug.log and ) and couldn't find errors due to the two variables.

          Show
          chirag24 Chirag Chandrahas Shetty (Inactive) added a comment - [~aloraine] Only one variable is removed from the code base ie: Expect App Name Variable. The code looks fine. I looked into the log files(debug.log and ) and couldn't find errors due to the two variables.
          chirag24 Chirag Chandrahas Shetty (Inactive) made changes -
          Status In Progress [ 3 ] Needs 1st Level Review [ 10005 ]
          chirag24 Chirag Chandrahas Shetty (Inactive) made changes -
          Assignee Chirag Chandrahas Shetty [ chirag24 ]
          ann.loraine Ann Loraine made changes -
          Sprint Winter 1 Dec 28 - Jan 8, Winter 2 Jan 11 - Jan 22, Winter 3 Jan 25 - Feb 5 [ 111, 112, 113 ] Winter 1 Dec 28 - Jan 8, Winter 2 Jan 11 - Jan 22, Winter 3 Jan 25 - Feb 5, Winter 4 Feb 8 - Feb 19 [ 111, 112, 113, 114 ]
          ann.loraine Ann Loraine made changes -
          Rank Ranked higher
          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 Sameer Shanbhag [ sameer ]

            People

            • Assignee:
              sameer Sameer Shanbhag (Inactive)
              Reporter:
              sameer Sameer Shanbhag (Inactive)
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: