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

appview: update state, ingester, middleware, and resolver for repo DID #1140

open opened by oyster.cafe targeting master from oyster.cafe/tangled-core: master
STACK 9
#1144 knotserver: add repo DID migration on startup
1
#13
#1143 appview, knotserver: add didAssign event and PDS rewrite on login
1
#13
#1142 spindle: add repo DID support in ingester, secrets, and xrpc handlers
0
#13
#1140 appview: update state, ingester, middleware, and resolver for repo DID
2
#13
#1138 appview: add repo_did columns, migrations, and model struct fields
1
#13
#1137 knotserver: DID-based repo resolution in internal, ingester, git, and router
0
#13
#1136 knotserver/xrpc: use repo DID resolution in all handlers
0
#13
#1135 knotserver: add repodid package, db migrations, and go-didplc dep
1
#13
#1134 lexicons, api/tangled: add repoDid field to all repo-scoped lexicons
2
#13
ABANDONED PULLS 2
#1141 appview: update repo, pulls, issues, and pages handlers for repo DID
2
#8
#1139 appview/db: update query functions for repo_did across all tables
0
#8
Labels

None yet.

assignee

None yet.

Participants 2
AT URI
at://did:plc:3fwecdnvtcscjnrx2p4n7alz/sh.tangled.repo.pull/3mgprvt2eon22
+3 -9
Interdiff #3 → #4
appview/ingester.go

This file has not been changed.

appview/middleware/middleware.go

This file has not been changed.

+1 -7
appview/reporesolver/resolver.go
··· 85 85 86 86 stats := repo.RepoStats 87 87 if stats == nil { 88 - var starCount int 89 - var starErr error 90 - if repo.RepoDid != "" { 91 - starCount, starErr = db.GetStarCountByRepoDid(rr.execer, repo.RepoDid, repoAt) 92 - } else { 93 - starCount, starErr = db.GetStarCount(rr.execer, repoAt) 94 - } 88 + starCount, starErr := db.GetStarCount(rr.execer, repo.RepoDid, repoAt) 95 89 if starErr != nil { 96 90 log.Println("failed to get star count for ", repoAt) 97 91 }
appview/state/git_http.go

This file has not been changed.

appview/state/knotstream.go

This file has not been changed.

appview/state/router.go

This file has not been changed.

+2 -2
appview/state/star.go
··· 81 81 return 82 82 } 83 83 84 - starCount, err := db.GetStarCount(s.db, subjectUri) 84 + starCount, err := db.GetStarCount(s.db, "", subjectUri) 85 85 if err != nil { 86 86 log.Println("failed to get star count for ", subjectUri) 87 87 } ··· 120 120 // this is not an issue, the firehose event might have already done this 121 121 } 122 122 123 - starCount, err := db.GetStarCount(s.db, subjectUri) 123 + starCount, err := db.GetStarCount(s.db, "", subjectUri) 124 124 if err != nil { 125 125 log.Println("failed to get star count for ", subjectUri) 126 126 return
appview/state/state.go

This file has not been changed.

appview/validator/label.go

This file has not been changed.

History

14 rounds 2 comments
sign up or login to add to the discussion
oyster.cafe submitted #13
diff interdiff
1 commit
expand
591b8c0e
appview: DID-based routing, state/handler/middleware updates
no conflicts, ready to merge
expand 0 comments
oyster.cafe submitted #12
diff interdiff
1 commit
expand
e33b9d68
appview: DID-based routing, state/handler/middleware updates
expand 0 comments
oyster.cafe submitted #11
diff interdiff
1 commit
expand
e425fb18
appview: DID-based routing, state/handler/middleware updates
expand 2 comments
boltless.me

appview/state/state.go:631 won't this eventually redirect to /{owner}/{reponame}?

oyster.cafe

wdym by eventually? I was thinking to keep this in, so that we don't simply error out if someone does decide to be clever and link to their git repo by repoDID, we should render the page anyway to reward them for being clever instead of punishing heh

oyster.cafe submitted #10
diff interdiff
1 commit
expand
ff34f7c0
appview: DID-based routing, state/handler/middleware updates
expand 0 comments
oyster.cafe submitted #9
diff interdiff
1 commit
expand
7bf3eb04
appview: DID-based routing, state/handler/middleware updates
expand 0 comments
oyster.cafe submitted #8
diff interdiff
1 commit
expand
e9243303
appview: update state, ingester, middleware, and resolver for repo DID
expand 0 comments
oyster.cafe submitted #7
diff interdiff
1 commit
expand
e9243303
appview: update state, ingester, middleware, and resolver for repo DID
expand 0 comments
oyster.cafe submitted #6
diff interdiff
1 commit
expand
aa2115d5
appview: update state, ingester, middleware, and resolver for repo DID
expand 0 comments
oyster.cafe submitted #5
diff interdiff
1 commit
expand
eefedc1a
appview: update state, ingester, middleware, and resolver for repo DID
expand 0 comments
oyster.cafe submitted #4
diff
1 commit
expand
bef1ed88
appview: update state, ingester, middleware, and resolver for repo DID
expand 0 comments
oyster.cafe submitted #3
diff interdiff
1 commit
expand
6d20b850
appview: update state, ingester, middleware, and resolver for repo DID
expand 0 comments
oyster.cafe submitted #2
diff interdiff
1 commit
expand
1c7e78e9
appview: update state, ingester, middleware, and resolver for repo DID
expand 0 comments
oyster.cafe submitted #1
diff interdiff
1 commit
expand
7806fbc0
appview: update state, ingester, middleware, and resolver for repo DID
expand 0 comments
oyster.cafe submitted #0
diff
1 commit
expand
8c8bd320
appview: update state, ingester, middleware, and resolver for repo DID
expand 0 comments