appsembler / edx-platform

Appsembler Tahoe fork of Open edX. Branches: `main` for Juniper and `hawthorn/main` for Hawthorn.
https://appsembler.com/products/tahoe/
GNU Affero General Public License v3.0
13 stars 15 forks source link

Check TPA pipeline for tahoe idp metadata if before added to UserProfile #1335

Closed bryanlandia closed 1 year ago

bryanlandia commented 1 year ago

Change description

TahoeUserMetadataProcessor try to get tahoe_idp_metadata from TPA pipeline if it's not found on the UserProfile (or in cache). This is needed for events that may fire before the UserProfile is created for a new user, or at least before the tahoe_idp pipeline step openedx.core.djangoapps.appsembler.tahoe_idp.tpa_pipeline.tahoe_idp_user_updates has added IdP metadata to UserProfile. This is specifically to deal with edx.course.enrollment.activated events which fire from the User.post_save Signal receiver when a learner is enrolled in a course before registering, receiving a CourseEnrollmentActivated object which will add the CourseEnrollment as soon as a User is created.

Type of change

Related issues

RED-3716

Checklists

Development

Security

Code review

github-actions[bot] commented 1 year ago

Checking git merge conflicts against https://github.com/edx/edx-platform.git

Comparing with open-release/nutmeg.master
Benchmark conflicts with main 292
Current conflicts 292
Summary Good work! No added conflicts.
Comparing with master
Benchmark conflicts with main 300
Current conflicts 300
Summary Good work! No added conflicts.
github-actions[bot] commented 1 year ago

Checking git merge conflicts against https://github.com/edx/edx-platform.git

Comparing with open-release/nutmeg.master
Benchmark conflicts with main 292
Current conflicts 292
Summary Good work! No added conflicts.
Comparing with master
Benchmark conflicts with main 300
Current conflicts 300
Summary Good work! No added conflicts.