appview: pulls: supply posthog client to PullsRouter · julien.rbrt.fr/tangled-core@2fcaf46

Monorepo for Tangled — https://tangled.org

appview: pulls: supply posthog client to PullsRouter

if the client is nil, posthog fails to enqueue a request, it crashes the
route and prevents redirect (as seen while creating pull comments and
pull requests)

Signed-off-by: oppiliappan <me@oppi.li>

oppi.li 9 months ago 2fcaf469 89dcba81

verified

+3 -1

2 changed files

expand all

unified split

appview

pulls

pulls.go

state

router.go

appview/pulls/pulls.go

··· 51 51 resolver *idresolver.Resolver, 52 52 db *db.DB, 53 53 config *config.Config, 54 + posthog posthog.Client, 54 55 ) *Pulls { 55 56 return &Pulls{ 56 57 oauth: oauth, ··· 59 60 idResolver: resolver, 60 61 db: db, 61 62 config: config, 63 + posthog: posthog, 62 64 } 63 65 } 64 66

+1 -1

appview/state/router.go

··· 178 178 } 179 179 180 180 func (s *State) PullsRouter(mw *middleware.Middleware) http.Handler { 181 - pulls := pulls.New(s.oauth, s.repoResolver, s.pages, s.idResolver, s.db, s.config) 181 + pulls := pulls.New(s.oauth, s.repoResolver, s.pages, s.idResolver, s.db, s.config, s.posthog) 182 182 return pulls.Router(mw) 183 183 } 184 184