Skip to content

Conversation

@mariajgrimaldi
Copy link
Contributor

@mariajgrimaldi mariajgrimaldi commented Dec 17, 2025

Description

Refactor the current username update flow to use a shared behavior class and base updater view and serializer. This enables supporting email updates separately via a different endpoint.

Testing instructions

  1. Create a user by calling {{domain}}/eox-core/api/v1/user/ with the following payload:
{
   "username": "{{username}}",
   "email": "{{email}}",
   "password": "p@ssword",
   "fullname": "To Be Deleted"
}
  1. Change their email by calling {{domain}}/eox-core/support-api/v1/user/replace-email/?username={{username}}
{
    "new_email": "changed_{{email}}"
}

I'll implement integration tests once the pipeline is fixed cause I tried running it locally but failed doing so.

Additional information

A client requested the ability to update a user's email address immediately after it is changed in their single sign-on (SSO) identity provider. This is required to maintain GDPR compliance, as communications must always be delivered to the correct and current email recipient. Currently, although the email can be updated via SSO, the change is not applied in the platform until the user logs in. This creates a risk where emails are sent to an outdated address if the user does not log in immediately, potentially resulting in non-compliance with GDPR data management standards.

Checklist for Merge

  • Tested in a remote environment
  • Updated documentation
  • Rebased master/main
  • Squashed commits

Refactor the current username update flow to use a shared behavior class and base updater view and serializer. This enables supporting email updates separately via a different endpoint.
@mariajgrimaldi mariajgrimaldi requested a review from a team as a code owner December 17, 2025 20:22
@mariajgrimaldi mariajgrimaldi marked this pull request as draft December 17, 2025 20:22
@mariajgrimaldi
Copy link
Contributor Author

@eduNEXT/heimdall this PR is still a work in progress, but we have not been able to get a single successful Tutor integration test run. Do you know if there is something wrong with the pipeline? Thanks in advance!

@magajh magajh self-requested a review December 18, 2025 14:03
@mariajgrimaldi mariajgrimaldi marked this pull request as ready for review December 26, 2025 11:56
@mariajgrimaldi
Copy link
Contributor Author

mariajgrimaldi commented Dec 26, 2025

@eduNEXT/heimdall: I opened the PR so we can start the review process although I still have to add some integration tests for the replace-email endpoint.

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.

2 participants