`small-user-avatars`, `repo-avatars` - Restore for organizations by Copilot · Pull Request #9278 · refined-github/refined-github · GitHub
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

Copilot AI commented Apr 25, 2026

Copy link
Copy Markdown
Contributor

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
Contributor

@SunsetTechuila SunsetTechuila marked this pull request as draft April 28, 2026 11:54
@SunsetTechuila

Copy link
Copy Markdown
Contributor

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
Contributor

You're right :(

@fregante

Copy link
Copy Markdown
Member

@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
Co-authored-by: fregante <me@fregante.com>
Co-authored-by: fregante <1402241+fregante@users.noreply.github.com>
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