Details

    • Type: Bug
    • Status: Done
    • Priority: Medium
    • Resolution: Done
    • Affects Version/s: None
    • Fix Version/s: Carry Over - Fall
    • Labels:
      None

      Description

      1. Register company return 200 status even company already exists.
      2. Return 404 when company & user not found in getCompany & getUser API
      3. By pass get company API , so need to pass userdid
      4. Make UserDid optional for creating user & get user details.

        Attachments

        1. CompanyAlreadyExists.png
          CompanyAlreadyExists.png
          85 kB
        2. Company Not Found.png
          Company Not Found.png
          203 kB
        3. GetCompany_2.jpg
          GetCompany_2.jpg
          258 kB
        4. getCompany.png
          getCompany.png
          258 kB

          Activity

          Hide
          rohan.khandave Rohan J Khandave (Inactive) added a comment -

          Below points will get consider into DMS-1149.

          3. By pass get company API , so need to pass userdid
          4. Make UserDid optional for creating user & get user details.

          Show
          rohan.khandave Rohan J Khandave (Inactive) added a comment - Below points will get consider into DMS-1149 . 3. By pass get company API , so need to pass userdid 4. Make UserDid optional for creating user & get user details.
          Hide
          rohan.khandave Rohan J Khandave (Inactive) added a comment -

          Remaining 2 points are covered under DMS-1204.

          Show
          rohan.khandave Rohan J Khandave (Inactive) added a comment - Remaining 2 points are covered under DMS-1204 .
          Hide
          rohan.khandave Rohan J Khandave (Inactive) added a comment -

          Please check deployed with DMS-1204

          Show
          rohan.khandave Rohan J Khandave (Inactive) added a comment - Please check deployed with DMS-1204
          Hide
          jayshree.nagpure Jayshree Nagpure (Inactive) added a comment - - edited

          Verified on Preproduction

          1. Register company return 200 status even company already exists - showing "Company already exist" - Pass
          2. Return 404 when company & user not found in getCompany & getUser API
          -get company - fail
          -getUser - pass(Showing user not found with 404 error)
          3. By pass get company API , so need to pass userdid - Pass- Incase of userdid is invalid showing company details.
          -But if Accountdid is incorrect showing Redhiglight, instead is should show 404 error with proper validation message
          4. Make UserDid optional for creating user & get user details - UserDid when not given- "{"Errors":[

          {"type":"Error","message":"Invalid authorization header","code":401}

          ]}"
          Enter dummy userDid - created user and get user details correctly.

          Refer screen capture for failed point.

          Re-opening this ticket

          Thanks,
          Jayshree

          Priya Dhamande Sachin Hingole Hrishikesh Deshpande Satya

          Show
          jayshree.nagpure Jayshree Nagpure (Inactive) added a comment - - edited Verified on Preproduction 1. Register company return 200 status even company already exists - showing "Company already exist" - Pass 2. Return 404 when company & user not found in getCompany & getUser API - get company - fail -getUser - pass(Showing user not found with 404 error) 3. By pass get company API , so need to pass userdid - Pass- Incase of userdid is invalid showing company details. - But if Accountdid is incorrect showing Redhiglight, instead is should show 404 error with proper validation message 4. Make UserDid optional for creating user & get user details - UserDid when not given- "{"Errors":[ {"type":"Error","message":"Invalid authorization header","code":401} ]}" Enter dummy userDid - created user and get user details correctly. Refer screen capture for failed point. Re-opening this ticket Thanks, Jayshree Priya Dhamande Sachin Hingole Hrishikesh Deshpande Satya
          Hide
          rohan.khandave Rohan J Khandave (Inactive) added a comment - - edited

          Hello Jayshree Nagpure,

          We checked this issue , this issue occurred due to large account did. This issue exists at CB API portal.

          We checked wrong account did with less characters and we get proper response.

          Please refer attached screen shot for same.

          Thanks,
          RohanK

          CC – Swapnil Pandhare , Priya Dhamande

          Show
          rohan.khandave Rohan J Khandave (Inactive) added a comment - - edited Hello Jayshree Nagpure , We checked this issue , this issue occurred due to large account did. This issue exists at CB API portal. We checked wrong account did with less characters and we get proper response. Please refer attached screen shot for same. Thanks, RohanK CC – Swapnil Pandhare , Priya Dhamande
          Hide
          jayshree.nagpure Jayshree Nagpure (Inactive) added a comment -

          Working fine
          Closing this ticket

          Show
          jayshree.nagpure Jayshree Nagpure (Inactive) added a comment - Working fine Closing this ticket

            People

            Assignee:
            rohan.khandave Rohan J Khandave (Inactive)
            Reporter:
            rohan.khandave Rohan J Khandave (Inactive)
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:

                Time Tracking

                Estimated:
                Original Estimate - 16h
                16h
                Remaining:
                Time Spent - 4.5h Remaining Estimate - 11.5h
                11.5h
                Logged:
                Time Spent - 4.5h Remaining Estimate - 11.5h
                4.5h