Skip to content

feat(UserManager): Add getters for avatar URLs - #60734

Merged
sorbaugh merged 2 commits into
masterfrom
feat/user/avatar-url-getters
May 28, 2026
Merged

feat(UserManager): Add getters for avatar URLs#60734
sorbaugh merged 2 commits into
masterfrom
feat/user/avatar-url-getters

Conversation

@provokateurin

Copy link
Copy Markdown
Member

No description provided.

@provokateurinprovokateurin added this to the Nextcloud 35 milestone May 26, 2026
@provokateurin
provokateurin requested a review from a team as a code ownerMay 26, 2026 13:51
@provokateurin
provokateurin requested review from ArtificialOwl, artonge, leftybournes and salmart-dev and removed request for a teamMay 26, 2026 13:51
@provokateurin

Copy link
Copy Markdown
MemberAuthor

/backport to stable34

@artongeartonge left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Why is this needed?

Comment threadlib/private/User/User.php Outdated
@provokateurin

Copy link
Copy Markdown
MemberAuthor

Why is this needed?

It's used at least twice in Unified Sharing, so I didn't want to duplicate the logic and thought that this could be useful for others as well.

Comment threadlib/private/User/LazyUser.php Outdated
Comment threadlib/public/IUser.php Outdated
Signed-off-by: provokateurin <kate@provokateurin.de>
Signed-off-by: provokateurin <kate@provokateurin.de>
@provokateurin
provokateurinforce-pushed the feat/user/avatar-url-getters branch from 5477b6a to 3ce12c8CompareMay 27, 2026 04:44
@provokateurinprovokateurin changed the title feat(User): Add getters for avatar URLsfeat(UserManager): Add getters for avatar URLsMay 27, 2026
@sorbaugh
sorbaugh merged commit 72f5eea into masterMay 28, 2026
207 of 217 checks passed
@sorbaugh
sorbaugh deleted the feat/user/avatar-url-getters branch May 28, 2026 13:51
@nextcloud-botnextcloud-bot mentioned this pull request Aug 11, 2026
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to reviewWaiting for reviewsenhancementpending documentationThis pull request needs an associated documentation update

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@provokateurin@artonge@come-nc@sorbaugh