Skip to content

feat: user syncing for AAF users (AAI-885) - #302

Merged
marius-mather merged 28 commits into
aaf-devfrom
feat/aaf-user-sync
Oct 19, 2026
Merged

marius-mather merged 28 commits into
aaf-devfrom
feat/aaf-user-sync

Conversation

@marius-mather

@marius-mather marius-mather commented Oct 18, 2026 •

Copy link
Copy Markdown
Collaborator

Description

AAI-885: update user syncing for AAF accounts.

Changes

  • Reworked user syncing to account for AAF users and the different account types we have now:
    • Auth0 database connection users
    • Auth0 database connection users, linked to an AAF account
    • AAF only users
  • Allow converting between account types during sync, where we can clearly identify the change.
    NB:: we expect account types to change through the backend's API endpoints, so this should
    not normally occur during sync - only there as a fallback.
  • Fix an issue where AAF users might not have their account type set.
  • Unit tests

Checklist

  • I have commented my code, particularly in hard-to-understand areas
  • I have added unit / integration tests that prove my fix is effective or that my feature works
  • I have run all tests locally and they pass
  • I have updated the documentation (if applicable)
  • For any new secrets, I have updated the shared spreadsheet and the GitHub Secrets.

Example sync results

Existing Auth0 account:

Screenshot 2026-10-16 at 3 00 46 pm

User sync showing account change:

Screenshot 2026-10-16 at 3 03 02 pm

Linked account after sync:

Screenshot 2026-10-16 at 3 03 23 pm

@amandazhuyilan amandazhuyilan left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good - good to add these changes in while we work on switching between AAF and non-AAF accounts

@marius-mather
marius-mather merged commit 1d29c87 into aaf-dev Oct 19, 2026
4 checks passed
@marius-mather
marius-mather deleted the feat/aaf-user-sync branch October 19, 2026 02:18
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