Skip to content

feat: replace enterprise username import in progress/views.py with pluggable_override - #38108

Merged
marlonkeating merged 4 commits into
masterfrom
pwnage101/ENT-11572
May 11, 2026
Merged

marlonkeating merged 4 commits into
masterfrom
pwnage101/ENT-11572

Conversation

@pwnage101

@pwnage101 pwnage101 commented Mar 5, 2026

Copy link
Copy Markdown
Contributor

Remove get_enterprise_learner_generic_name import. Add obfuscated_username function decorated with @pluggable_override('OVERRIDE_COURSE_HOME_PROGRESS_USERNAME') returning None by default.

ENT-11572


Related:

…uggable_override

Remove get_enterprise_learner_generic_name import. Add obfuscated_username function decorated
with @pluggable_override('OVERRIDE_COURSE_HOME_PROGRESS_USERNAME') returning None by default.

ENT-11572
Comment thread lms/djangoapps/course_home_api/progress/views.py Outdated
@marlonkeating
marlonkeating marked this pull request as ready for review May 4, 2026 17:07

@pwnage101 pwnage101 left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! But remember DO NOT MERGE until this PR has been updated with a new version of edx-enterprise containing the override implementation.

@marlonkeating
marlonkeating merged commit 1498c8d into master May 11, 2026
46 checks passed
@marlonkeating
marlonkeating deleted the pwnage101/ENT-11572 branch May 11, 2026 14:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enterprise An enterprise-related change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants