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.

@provokateurin provokateurin added this to the Nextcloud 35 milestone May 26, 2026
@provokateurin provokateurin requested a review from a team as a code owner May 26, 2026 13:51
@provokateurin provokateurin requested review from ArtificialOwl, artonge, leftybournes and salmart-dev and removed request for a team May 26, 2026 13:51
@provokateurin
Copy link
Copy Markdown
Member Author

/backport to stable34

Copy link
Copy Markdown
Collaborator

@artonge artonge left a comment

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 thread lib/private/User/User.php Outdated
@provokateurin
Copy link
Copy Markdown
Member Author

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 thread lib/private/User/LazyUser.php Outdated
Comment thread lib/public/IUser.php Outdated
Signed-off-by: provokateurin <kate@provokateurin.de>
Signed-off-by: provokateurin <kate@provokateurin.de>
@provokateurin provokateurin force-pushed the feat/user/avatar-url-getters branch from 5477b6a to 3ce12c8 Compare May 27, 2026 04:44
@provokateurin provokateurin changed the title feat(User): Add getters for avatar URLs feat(UserManager): Add getters for avatar URLs May 27, 2026
@provokateurin provokateurin requested review from artonge and come-nc May 27, 2026 04:49
@sorbaugh sorbaugh merged commit 72f5eea into master May 28, 2026
207 of 217 checks passed
@sorbaugh sorbaugh deleted the feat/user/avatar-url-getters branch May 28, 2026 13:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews enhancement pending documentation This pull request needs an associated documentation update

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants