Student Success Plan
  1. Student Success Plan
  2. SSP-1817

External Person Sync Doesn't Sync Photo URL

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Critical Critical
    • Resolution: Fixed
    • Affects Version/s: SSP 2.0.0, SSP 2.1.0, SSP 2.0.1
    • Fix Version/s: SSP 2.1.0, SSP 2.0.1
    • Component/s: External Data
    • Labels:
    • Environment:
      All CI's and local builds.

      Description

      Scenario:

      1. Setup a student record in external_person with a photo url
      2. Log in to SSP as a coach and add a student
      3. External sync the created student such that they are imported into person table
      4. View the picture (one will not show)
      5. View the person table for that record and you should find nothing for the photo url.

      Issue: The photo url is not syncd into the person table.

        Activity

        Hide
        Jason Elwood added a comment -

        Assigning this to Dan as a first pass. This was reported by a NC implementer.

        Show
        Jason Elwood added a comment - Assigning this to Dan as a first pass. This was reported by a NC implementer.
        Hide
        Paul Spaude added a comment -

        Photo url wasn't in the list of items syncd from external_person to person models due to the fact the external_person model did not contain a representation for that column. Added photoUrl to the external_person model and to the person sync service.

        Show
        Paul Spaude added a comment - Photo url wasn't in the list of items syncd from external_person to person models due to the fact the external_person model did not contain a representation for that column. Added photoUrl to the external_person model and to the person sync service.
        Hide
        Paul Spaude added a comment -

        Added photourl to external_person model and the sync task. Tested the sync with external students.

        Show
        Paul Spaude added a comment - Added photourl to external_person model and the sync task. Tested the sync with external students.
        Hide
        Dan McCallum added a comment -

        Reopening to remove 2.0.1 fix version

        Show
        Dan McCallum added a comment - Reopening to remove 2.0.1 fix version
        Hide
        Dan McCallum added a comment -

        Ready for test in master (2.1.0). Has not been merged to rel-2-0-patches yet.

        Show
        Dan McCallum added a comment - Ready for test in master (2.1.0). Has not been merged to rel-2-0-patches yet.
        Hide
        Dan McCallum added a comment -

        Jim, can you please cherry-pick into rel-2-0-patches and test.

        When you start, please mark the ticket In Progress.

        Once done, please add the SSP 2.0.1 fix version and set to Resolved. Please leave the merge-to-2-0 label until it has been QA'd.

        Show
        Dan McCallum added a comment - Jim, can you please cherry-pick into rel-2-0-patches and test. When you start, please mark the ticket In Progress. Once done, please add the SSP 2.0.1 fix version and set to Resolved. Please leave the merge-to-2-0 label until it has been QA'd.
        Hide
        Dan McCallum added a comment -

        Reopening for cherry-pick into rel-2-0-patches

        Show
        Dan McCallum added a comment - Reopening for cherry-pick into rel-2-0-patches
        Hide
        James T. Stanley added a comment - - edited

        Patch has been uploaded to rel-2-0-patches

        Show
        James T. Stanley added a comment - - edited Patch has been uploaded to rel-2-0-patches

          People

          • Assignee:
            James T. Stanley
            Reporter:
            Paul Spaude
          • Votes:
            0 Vote for this issue
            Watchers:
            5 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Time Tracking

              Estimated:
              Original Estimate - Not Specified
              Not Specified
              Remaining:
              Remaining Estimate - 0 minutes
              0m
              Logged:
              Time Spent - 3 hours
              3h