tangled.org / core
Monorepo for Tangled tangled.org
fork atom

appview/pages: upload and render avatar #894

open
opened by anirudh.fi targeting master from icy/tolqpt
STACK 6
#896 appview/pages: render a placeholder avatar when there's no did
0
#1
#894 appview/pages: upload and render avatar
0
#2
#893 avatar: fetch tangled avatar before falling back to bsky & random color
0
#2
#892 appview/state: handle profile pic uploads
3/3
0
#2
#891 appview/{db,models}: add avatar to profile
3/3
0
#2
#890 lexicons,api: add avatar field to profile lexicon
3/3
0
#2
0
by anirudh.fi 0 comments
expand 1 commit
e6206e1b
appview/pages: upload and render avatar
build.yml
fmt.yml
test.yml
sign up or login to add to the discussion
1
by anirudh.fi 2 comments
expand 1 commit
afcdcfe8
appview/pages: upload and render avatar
oppi.li
  • profileAvatarUrl is only used where models.Profile is available, is the uploaded profile picture not rendered across the board? (the previous PR in the stack implies otherwise, in which case, why do we need any special logic for avatar URLs on the appview?)
  • this resolves handle again, we already have a resolved identity in ident
  • there is repetition between ProfileAvatarUrl and AvatarUrl (the signature calculation logic is duplicated)
anirudh.fi

profileAvatarUrl is only used where models.Profile is available, is the uploaded profile picture not rendered across the board? (the previous PR in the stack implies otherwise, in which case, why do we need any special logic for avatar URLs on the appview?)

it exists to construct the blob url to to fetch from the pds. the avatar service merely checks if a blob url has been provided (if not, fallback to bsky). i could clean this up though, i didn't particularly like it myself.

build.yml
fmt.yml
test.yml
sign up or login to add to the discussion
2
by anirudh.fi 0 comments
expand 1 commit
fa732f2b
appview/pages: upload and render avatar
no conflicts, ready to merge
sign up or login to add to the discussion
Labels

None yet.

assignee

None yet.

Participants 2
AT URI
at://did:plc:hwevmowznbiukdf6uk5dwrrq/sh.tangled.repo.pull/3m7znwnzq6k22