Skip to content

Add crate logos-accounts#102

Merged
jazzz merged 1 commit into
mainfrom
jazzz/account_ext
May 20, 2026
Merged

Add crate logos-accounts#102
jazzz merged 1 commit into
mainfrom
jazzz/account_ext

Conversation

@jazzz
Copy link
Copy Markdown
Collaborator

@jazzz jazzz commented May 19, 2026

Accounts were introduced in a previous PR however they were local to the core/conversations. This PR moves Accounts to their own crate so they can be re-used in other services.

Changes:

  • Adds IdentityProvider trait to service_traits so it is available for external implementors.
  • Adds logos-account crate, as it need to be accessible by applications to administer user accounts. This change while perhaps permature, establishes it as an external entity.
  • Updated IdentityProvider::friendly_name -> display_name as suggested by @osmaczko .

Future Work:

  • Update core::context to accept an external IdentityProvider impl on construction.
  • Update InboxV2 to use this

@jazzz
Copy link
Copy Markdown
Collaborator Author

jazzz commented May 19, 2026

This PR is dependent on #101 being merged first.

@jazzz jazzz force-pushed the jazzz/account_ext branch from 754a192 to 402abb5 Compare May 20, 2026 20:36
@jazzz jazzz force-pushed the jazzz/account_ext branch from 402abb5 to 808c0de Compare May 20, 2026 20:42
@jazzz jazzz merged commit fa68714 into main May 20, 2026
7 checks passed
@jazzz jazzz deleted the jazzz/account_ext branch May 20, 2026 21:11
@jazzz jazzz mentioned this pull request May 22, 2026
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants