Monorepo for Tangled tangled.org

appview: migrate handlers to GetMultiAccountUser #943

merged opened by lewis.moe targeting master from lewis.moe/tangled-core: master
Labels

None yet.

assignee

None yet.

Participants 2
AT URI
at://did:plc:3fwecdnvtcscjnrx2p4n7alz/sh.tangled.repo.pull/3mc2mfptapt22
+4 -4
Interdiff #1 #2
appview/issues/issues.go

This file has not been changed.

appview/knots/knots.go

This file has not been changed.

appview/labels/labels.go

This file has not been changed.

appview/middleware/middleware.go

This file has not been changed.

appview/notifications/notifications.go

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

+4 -4
appview/pipelines/pipelines.go
··· 70 70 } 71 71 72 72 func (p *Pipelines) Index(w http.ResponseWriter, r *http.Request) { 73 - user := p.oauth.GetMultiAccountUser(r) 73 + user := p.oauth.GetUser(r) 74 74 l := p.logger.With("handler", "Index") 75 75 76 76 f, err := p.repoResolver.Resolve(r) 77 77 } 78 78 79 79 func (p *Pipelines) Index(w http.ResponseWriter, r *http.Request) { 80 - user := p.oauth.GetUser(r) 80 + user := p.oauth.GetMultiAccountUser(r) 81 81 l := p.logger.With("handler", "Index") 82 82 83 83 f, err := p.repoResolver.Resolve(r) ··· 99 99 } 100 100 101 101 func (p *Pipelines) Workflow(w http.ResponseWriter, r *http.Request) { 102 - user := p.oauth.GetMultiAccountUser(r) 102 + user := p.oauth.GetUser(r) 103 103 l := p.logger.With("handler", "Workflow") 104 104 105 105 f, err := p.repoResolver.Resolve(r) 106 106 } 107 107 108 108 func (p *Pipelines) Workflow(w http.ResponseWriter, r *http.Request) { 109 - user := p.oauth.GetUser(r) 109 + user := p.oauth.GetMultiAccountUser(r) 110 110 l := p.logger.With("handler", "Workflow") 111 111 112 112 f, err := p.repoResolver.Resolve(r)
appview/pulls/pulls.go

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

appview/repo/artifact.go

This file has not been changed.

appview/repo/blob.go

This file has not been changed.

appview/repo/branches.go

This file has not been changed.

appview/repo/compare.go

This file has not been changed.

appview/repo/index.go

This file has not been changed.

appview/repo/log.go

This file has not been changed.

appview/repo/repo.go

This file has not been changed.

appview/repo/settings.go

This file has not been changed.

appview/repo/tags.go

This file has not been changed.

appview/repo/tree.go

This file has not been changed.

appview/reporesolver/resolver.go

This file has not been changed.

appview/settings/settings.go

This file has not been changed.

appview/spindles/spindles.go

This file has not been changed.

appview/state/follow.go

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

appview/state/gfi.go

This file has not been changed.

appview/state/profile.go

This file has not been changed.

appview/state/reaction.go

This file has not been changed.

appview/state/star.go

This file has not been changed.

appview/state/state.go

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

appview/strings/strings.go

This file has not been changed.

History

3 rounds 2 comments
sign up or login to add to the discussion
lewis.moe submitted #2
1 commit
expand
appview: migrate handlers to GetMultiAccountUser
expand 0 comments
pull request successfully merged
1 commit
expand
appview: migrate handlers to GetMultiAccountUser
expand 0 comments
1 commit
expand
appview: migrate handlers to GetMultiAccountUser
expand 2 comments

can you explain why we are switching to GetMultiAccountUser?

oh wait, sorry I get it now. we need full account list for rendering. Makes sense.

Hopefully #800 can make this better...