Skip to content

feat: remove change_enterprise_user_username management command#228

Open
marlonkeating wants to merge 1 commit intorelease-ulmofrom
mkeating/ENT-11565
Open

feat: remove change_enterprise_user_username management command#228
marlonkeating wants to merge 1 commit intorelease-ulmofrom
mkeating/ENT-11565

Conversation

@marlonkeating
Copy link
Copy Markdown

ENT-11565, sister PR to openedx#38310

Deletes the change_enterprise_user_username management command and its tests from openedx-platform. This command is being removed as part of a wider effort to migrate enterprise functionality to edx-enterprise, but in this case as the command is no longer in active use, we are simply removing it.

feat: remove change_enterprise_user_username management command
Copilot AI review requested due to automatic review settings April 10, 2026 17:17
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Removes an unused Enterprise-related Django management command (change_enterprise_user_username) from the student app, as part of migrating enterprise functionality out of openedx-platform (ENT-11565).

Changes:

  • Deleted the change_enterprise_user_username management command implementation.
  • Deleted the corresponding management command test module.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
common/djangoapps/student/management/commands/change_enterprise_user_username.py Removes the management command that updated enterprise-linked user usernames.
common/djangoapps/student/management/tests/test_change_enterprise_user_username.py Removes tests that exercised the deleted management command.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants