Monorepo for Tangled tangled.org

appview/{oauth,pages}: remove Pds field from oauth.User #1028

deleted opened by boltless.me targeting master from sl/uvpzuszrulvq

The Pds field is only used to render it on settings/profile page. We can get user PDS from DID instead.

Signed-off-by: Seongmin Lee git@boltless.me

Labels

None yet.

assignee

None yet.

Participants 1
AT URI
at://did:plc:xasnlahkri4ewmbuzly2rlc5/sh.tangled.repo.pull/3mdcq5ouyxq22
+2 -2
Interdiff #2 #3
appview/oauth/accounts.go

This file has not been changed.

appview/oauth/accounts_test.go

This file has not been changed.

appview/oauth/oauth.go

This patch was likely rebased, as context lines do not match.

appview/pages/funcmap.go

This patch was likely rebased, as context lines do not match.

+2 -2
appview/pages/templates/user/settings/profile.html
··· 34 34 </div> 35 35 <div class="flex flex-col gap-1 p-4"> 36 36 <span class="text-sm text-gray-500 dark:text-gray-400">Personal Data Server (PDS)</span> 37 - <span class="font-bold">{{ .LoggedInUser.Pds }}</span> 37 + <span class="font-bold">{{ resolvePds .LoggedInUser.Did }}</span> 38 38 </div> 39 39 </div> 40 40 </div> ··· 54 54 <span>Personal Data Server (PDS)</span> 55 55 </div> 56 56 <span class="font-bold"> 57 - {{ resolvePds .LoggedInUser.Did }} 57 + {{ .LoggedInUser.Pds }} 58 58 </span> 59 59 </div> 60 60 </div>

History

5 rounds 0 comments
sign up or login to add to the discussion
1 commit
expand
appview/{oauth,pages}: remove Pds field from oauth.User
1/3 failed, 2/3 success
expand
expand 0 comments
This pull has been deleted (possibly by jj abandon or jj squash)
1 commit
expand
appview/{oauth,pages}: remove Pds field from oauth.User
3/3 success
expand
expand 0 comments
1 commit
expand
appview/{oauth,pages}: remove Pds field from oauth.User
1/3 failed, 2/3 success
expand
expand 0 comments
1 commit
expand
appview/{oauth,pages}: remove Pds field from oauth.User
1/3 failed, 2/3 success
expand
expand 0 comments
1 commit
expand
appview/{oauth,pages}: remove Pds field from oauth.User
1/3 failed, 2/3 success
expand
expand 0 comments