Skip to content

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

Open
pwnage101 wants to merge 4 commits intomasterfrom
pwnage101/ENT-11572
Open

feat: replace enterprise username import in progress/views.py with pluggable_override#38108
pwnage101 wants to merge 4 commits intomasterfrom
pwnage101/ENT-11572

Conversation

@pwnage101
Copy link
Copy Markdown
Contributor

@pwnage101 pwnage101 commented Mar 5, 2026

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
Copy link
Copy Markdown
Contributor Author

@pwnage101 pwnage101 left a comment

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.

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