Uploaded image for project: 'WORKTERRA'
  1. WORKTERRA
  2. WT-11800

City of Redding - report save failure Production

    Details

    • Type: Bug
    • Status: Production Complete
    • Priority: Medium
    • Resolution: Bug Fixed
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: BenAdmin
    • Labels:
    • Environment:
      Production
    • Bug Severity:
      Medium
    • Module:
      BenAdmin - Report
    • Reported by:
      CareerBuilder
    • Company:
      City of Redding
    • Item State:
      Production Complete - Closed
    • Sprint:
      WT- Sprint 43

      Description

      Hello Dev Team

      When trying to save a report for City of Redding, I am receiving a failure notification. I am able to run the report and get results but not save it. Below are print screens of the report being run.

        Attachments

        1. City of Redding.docx
          658 kB
        2. Old report save message.png
          Old report save message.png
          209 kB
        3. WT-11800_CodeReviewChecklist.docx
          18 kB
        4. WT-11800.xls
          7 kB

          Issue Links

            Activity

            Hide
            rakesh.shahane Rakesh Shahane (Inactive) added a comment -

            Hi darryn.carter,
            We did not found the report with name enrollment as provided in screenshots on production, However we have also checked for the other reports on same company and not found any issue while saving.

            Could you please provide more details on this?

            CC-Umesh Kadam, Amruta Lohiya, Satya, Samir

            Show
            rakesh.shahane Rakesh Shahane (Inactive) added a comment - Hi darryn.carter , We did not found the report with name enrollment as provided in screenshots on production, However we have also checked for the other reports on same company and not found any issue while saving. Could you please provide more details on this? CC- Umesh Kadam , Amruta Lohiya , Satya , Samir
            Hide
            darryn.carter darryn.carter added a comment -

            Hello Dev Team

            The Error occurs when trying to save. You would need to try and create this report and save it to see the error.

            Show
            darryn.carter darryn.carter added a comment - Hello Dev Team The Error occurs when trying to save. You would need to try and create this report and save it to see the error.
            Hide
            rakesh.shahane Rakesh Shahane (Inactive) added a comment -

            Hi darryn.carter,
            We have checked this and found that report with name Enrollment is already exist, Due to this error occurs while trying to save.

            Work Around: Please create report with other name.

            Prasanna Karlekar We need to add proper message in case of report with same name exist, Instead of Report save failure message.

            CC-Umesh Kadam, Amruta Lohiya, Satya, Samir

            Show
            rakesh.shahane Rakesh Shahane (Inactive) added a comment - Hi darryn.carter , We have checked this and found that report with name Enrollment is already exist, Due to this error occurs while trying to save. Work Around: Please create report with other name. Prasanna Karlekar We need to add proper message in case of report with same name exist, Instead of Report save failure message. CC- Umesh Kadam , Amruta Lohiya , Satya , Samir
            Hide
            prasanna Prasanna Karlekar (Inactive) added a comment -

            Hi darryn.carter,

            We will try to schedule this change for proper error message in upcoming sprint starting from 06-Nov-2017.

            Thank you!

            cc: Gaurav Sodani, [~parmeshwar.jumbade]

            Show
            prasanna Prasanna Karlekar (Inactive) added a comment - Hi darryn.carter , We will try to schedule this change for proper error message in upcoming sprint starting from 06-Nov-2017. Thank you! cc: Gaurav Sodani , [~parmeshwar.jumbade]
            Hide
            parmeshwar.jumbad Parmeshwar Jumbad (Inactive) added a comment - - edited

            Below are the 3C's:

            Concern: Receiving a failure notification while save but able to preview report.

            Cause: If entered report name same as any of saved report it was showing failure notifications instead of meaning full message. This issue was exist in case of Standard and dynamic reports.
            Along with that we found few more scenarios which are listed below:
            1. If we are saving report in standard report base type and any other report with same name in dynamic report in this case also it showing failure notifications.
            2. While clone standard report it was checking duplication of report only already created report sub-type if there is exist same name in standard report and another sub type then it getting save.

            Correction: We have added user readable message in case of duplication of report name into respective type.
            Now report name duplication will work as following ways:
            1. When we are saving or cloning Standard/Dynamic report it will only check duplicate name in current selected report base type reports. This will not check report sub-type because we are showing these reports into a single grid based on the report base type (i.e. Standard/Dynamic).
            2. Static report will be work as of now - When we are saving or cloning static report it will only check duplicate name in current selected report type (i.e. Base Type + report sub type) . This will check report sub-type because we are showing these reports into a separate grid based on the report base type and sub type.

            Affected Areas: Reports (All Level and All types of reports)
            Affected Files:
            Areas/Report/Controllers/StandardReportsController/StandardReportsController.cs
            Areas/Report/Models/CloneReportModel/CloneReportModel.cs
            Areas/Report/Models/StandardReportsModel/StandardReportsModel.cs

            Tested Scenarios:
            1. Verified report save functionality (i.e. Add/Edit) for all types and all levels
            1. Verified report clone functionality for all types and all levels
            2. Checked report save (i.e. Add/Edit) with & without same report name in different report base type as well as sub type.
            4. Checked report clone functionality with & without same report name in different report base type as well as sub type.

            Known issue in New UI:
            1. Loader not appearing before success or failure popup. This need to be fixed because end user could not understand whether action started or not. In old UI Loader appear after click on Clone button.

            Note: We have only checked-in changes in Old UI and will do same into new UI after it ready for stage.

            CC: Prasanna Karlekar Amruta Lohiya Gaurav Sodani

            Show
            parmeshwar.jumbad Parmeshwar Jumbad (Inactive) added a comment - - edited Below are the 3C's : Concern : Receiving a failure notification while save but able to preview report. Cause : If entered report name same as any of saved report it was showing failure notifications instead of meaning full message. This issue was exist in case of Standard and dynamic reports. Along with that we found few more scenarios which are listed below: 1. If we are saving report in standard report base type and any other report with same name in dynamic report in this case also it showing failure notifications. 2. While clone standard report it was checking duplication of report only already created report sub-type if there is exist same name in standard report and another sub type then it getting save. Correction : We have added user readable message in case of duplication of report name into respective type. Now report name duplication will work as following ways: 1. When we are saving or cloning Standard/Dynamic report it will only check duplicate name in current selected report base type reports. This will not check report sub-type because we are showing these reports into a single grid based on the report base type (i.e. Standard/Dynamic). 2. Static report will be work as of now - When we are saving or cloning static report it will only check duplicate name in current selected report type (i.e. Base Type + report sub type) . This will check report sub-type because we are showing these reports into a separate grid based on the report base type and sub type. Affected Areas : Reports (All Level and All types of reports) Affected Files : Areas/Report/Controllers/StandardReportsController/StandardReportsController.cs Areas/Report/Models/CloneReportModel/CloneReportModel.cs Areas/Report/Models/StandardReportsModel/StandardReportsModel.cs Tested Scenarios : 1. Verified report save functionality (i.e. Add/Edit) for all types and all levels 1. Verified report clone functionality for all types and all levels 2. Checked report save (i.e. Add/Edit) with & without same report name in different report base type as well as sub type. 4. Checked report clone functionality with & without same report name in different report base type as well as sub type. Known issue in New UI : 1. Loader not appearing before success or failure popup. This need to be fixed because end user could not understand whether action started or not. In old UI Loader appear after click on Clone button. Note : We have only checked-in changes in Old UI and will do same into new UI after it ready for stage. CC: Prasanna Karlekar Amruta Lohiya Gaurav Sodani
            Hide
            venkatesh.pujari Venkatesh Pujari (Inactive) added a comment -

            Hi [~parmeshwar.jumbade],

            Tested this fix on City Of Durham For Hspl company on LB. Working fine as expected. Now correct message is being displayed for duplicate name on Add or Edit report in Standard,Dynamic and Static reports. This is ready for Stage.

            Hrishikesh Deshpande,Sachin Hingole

            Thanks,
            Venkatesh

            Show
            venkatesh.pujari Venkatesh Pujari (Inactive) added a comment - Hi [~parmeshwar.jumbade] , Tested this fix on City Of Durham For Hspl company on LB. Working fine as expected. Now correct message is being displayed for duplicate name on Add or Edit report in Standard,Dynamic and Static reports. This is ready for Stage. Hrishikesh Deshpande , Sachin Hingole Thanks, Venkatesh
            Hide
            venkatesh.pujari Venkatesh Pujari (Inactive) added a comment -

            Hi [~parmeshwar.jumbade],

            Verified this fix on Stage on AFI For Hspl company. The system is displaying report save failure on Edit of a report. When we edit any standard report and enter the duplicate name then the system is displaying " Report Save Failure " . Please could you look into this and reassign this ticket once fixed.

            REOPEN on Stage

            Sachin Hingole

            Thanks,
            Venkatesh

            Show
            venkatesh.pujari Venkatesh Pujari (Inactive) added a comment - Hi [~parmeshwar.jumbade] , Verified this fix on Stage on AFI For Hspl company. The system is displaying report save failure on Edit of a report. When we edit any standard report and enter the duplicate name then the system is displaying " Report Save Failure " . Please could you look into this and reassign this ticket once fixed. REOPEN on Stage Sachin Hingole Thanks, Venkatesh
            Hide
            parmeshwar.jumbad Parmeshwar Jumbad (Inactive) added a comment - - edited

            Hi Venkatesh Pujari,

            Below are the 3C's:

            Concern: While save report with duplicate name it showing 'report failure message'.
            Cause: Changes reverted by my side while check-in on the same day of stage deployment.
            Correction: Reverted changes committed into stage environment code branch.

            CC: Amruta Lohiya Gaurav Sodani Satya Rakesh Roy

            Show
            parmeshwar.jumbad Parmeshwar Jumbad (Inactive) added a comment - - edited Hi Venkatesh Pujari , Below are the 3C's : Concern : While save report with duplicate name it showing 'report failure message'. Cause : Changes reverted by my side while check-in on the same day of stage deployment. Correction : Reverted changes committed into stage environment code branch. CC: Amruta Lohiya Gaurav Sodani Satya Rakesh Roy
            Hide
            venkatesh.pujari Venkatesh Pujari (Inactive) added a comment -

            Hi [~parmeshwar.jumbade],

            Tested this issue on Stage on AFI For Hspl company. Clone issue exists while cloning a report with that report name only.

            Reopen

            Thanks,
            Venkatesh

            Show
            venkatesh.pujari Venkatesh Pujari (Inactive) added a comment - Hi [~parmeshwar.jumbade] , Tested this issue on Stage on AFI For Hspl company. Clone issue exists while cloning a report with that report name only. Reopen Thanks, Venkatesh
            Hide
            parmeshwar.jumbad Parmeshwar Jumbad (Inactive) added a comment - - edited

            Hi Venkatesh Pujari,

            As discuss with Amruta Lohiya, this one is a special case and exists on all environments and we can consider this while report UI regression points.

            We will discuss same with Satya will check if this can be move on production environment.

            CC: Amruta Lohiya Gaurav Sodani Satya Rakesh Roy

            Show
            parmeshwar.jumbad Parmeshwar Jumbad (Inactive) added a comment - - edited Hi Venkatesh Pujari , As discuss with Amruta Lohiya , this one is a special case and exists on all environments and we can consider this while report UI regression points. We will discuss same with Satya will check if this can be move on production environment. CC: Amruta Lohiya Gaurav Sodani Satya Rakesh Roy
            Hide
            venkatesh.pujari Venkatesh Pujari (Inactive) added a comment -

            Hi [~parmeshwar.jumbade],

            As per discussion the reported issue will be taken in Regression part. This is ready for Production.

            Thanks,
            Venkatesh

            Show
            venkatesh.pujari Venkatesh Pujari (Inactive) added a comment - Hi [~parmeshwar.jumbade] , As per discussion the reported issue will be taken in Regression part. This is ready for Production. Thanks, Venkatesh
            Hide
            venkatesh.pujari Venkatesh Pujari (Inactive) added a comment -

            Hi [~parmeshwar.jumbade],

            Tested this fix on JE Dunn For Hspl company on Production. Working fine as expected. Now correct message is being displayed for duplicate name on Add,clone or Edit report in Standard,Dynamic and Static reports.

            Hrishikesh Deshpande,Sachin Hingole

            Thanks,
            Venkatesh

            Show
            venkatesh.pujari Venkatesh Pujari (Inactive) added a comment - Hi [~parmeshwar.jumbade] , Tested this fix on JE Dunn For Hspl company on Production. Working fine as expected. Now correct message is being displayed for duplicate name on Add,clone or Edit report in Standard,Dynamic and Static reports. Hrishikesh Deshpande , Sachin Hingole Thanks, Venkatesh
            Hide
            venkatesh.pujari Venkatesh Pujari (Inactive) added a comment -

            Hi darryn.carter,

            The issue has been resolved and also tested on Production. Please could you verify and close this ticket.

            Sachin Hingole,Hrishikesh Deshpande

            Thanks,
            Venkatesh

            Show
            venkatesh.pujari Venkatesh Pujari (Inactive) added a comment - Hi darryn.carter , The issue has been resolved and also tested on Production. Please could you verify and close this ticket. Sachin Hingole , Hrishikesh Deshpande Thanks, Venkatesh

              People

              Assignee:
              darryn.carter darryn.carter
              Reporter:
              darryn.carter darryn.carter
              Developer:
              Parmeshwar Jumbad (Inactive)
              QA:
              Venkatesh Pujari (Inactive)
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:
                Dev Due Date:
                Production Due Date:

                  Time Tracking

                  Estimated:
                  Original Estimate - 4h Original Estimate - 4h
                  4h
                  Remaining:
                  Remaining Estimate - 0h
                  0h
                  Logged:
                  Time Spent - 16.75h
                  16.75h