Skip to content

small-user-avatars, repo-avatars - Restore for organizations#9278

Merged
fregante merged 8 commits into
mainfrom
copilot/fix-small-user-avatars
Apr 30, 2026
Merged

small-user-avatars, repo-avatars - Restore for organizations#9278
fregante merged 8 commits into
mainfrom
copilot/fix-small-user-avatars

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 25, 2026

Copilot AI changed the title [WIP] Fix small user avatars and repo avatars not working fix: use /username.png for avatars instead of avatars.githubusercontent.com/username Apr 25, 2026
Copilot AI requested a review from fregante April 25, 2026 11:52
Comment thread source/github-helpers/get-user-avatar.ts Outdated
@fregante fregante changed the title fix: use /username.png for avatars instead of avatars.githubusercontent.com/username Restore support for certain organization avatars Apr 25, 2026
@fregante fregante added the bug label Apr 25, 2026
Copilot AI changed the title Restore support for certain organization avatars fix: use /username.png for avatars instead of avatars.githubusercontent.com/username Apr 25, 2026
Copilot AI requested a review from fregante April 25, 2026 11:59
@github-actions github-actions Bot changed the title fix: use /username.png for avatars instead of avatars.githubusercontent.com/username Use /username.png for avatars instead of avatars.githubusercontent.com/username Apr 25, 2026
@fregante fregante changed the title Use /username.png for avatars instead of avatars.githubusercontent.com/username Restore support for certain organization avatars Apr 25, 2026
@SunsetTechuila SunsetTechuila changed the title Restore support for certain organization avatars small-user-avatars, repo-avatars - Restore for organizations Apr 28, 2026
@SunsetTechuila SunsetTechuila marked this pull request as ready for review April 28, 2026 11:41
@SunsetTechuila
Copy link
Copy Markdown
Member

More and more organizations are being affected :(

@SunsetTechuila SunsetTechuila marked this pull request as draft April 28, 2026 11:54
@SunsetTechuila
Copy link
Copy Markdown
Member

This is very slow. Let's cache the final URL

@fregante
Copy link
Copy Markdown
Member

You can't know the final URL without making two requests, so it would only make it slower.

This reverts commit 18816b2.
@SunsetTechuila
Copy link
Copy Markdown
Member

You're right :(

@fregante
Copy link
Copy Markdown
Member

So mergeable again? I plan to release 26.5.1 so I'll be reviewing PRs this week

@SunsetTechuila SunsetTechuila marked this pull request as ready for review April 28, 2026 19:54
@fregante fregante merged commit 6381050 into main Apr 30, 2026
10 checks passed
@SunsetTechuila SunsetTechuila deleted the copilot/fix-small-user-avatars branch April 30, 2026 09:38
Copilot AI added a commit that referenced this pull request Apr 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Development

Successfully merging this pull request may close these issues.

small-user-avatars and repo-avatars stopped working with some organizations

3 participants