tangled.org / core

Monorepo for Tangled tangled.org

appview: update repo, pulls, issues, and pages handlers for repo DID #1141

deleted opened by

oyster.cafe 1 week ago targeting master from oyster.cafe/tangled-core: master

Signed-off-by: Lewis lewis@tangled.org

STACK 9

#1144 knotserver: add repo DID migration on startup

#1143 appview, knotserver: add didAssign event and PDS rewrite on login

#1142 spindle: add repo DID support in ingester, secrets, and xrpc handlers

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

#1138 appview: add repo_did columns, migrations, and model struct fields

#1137 knotserver: DID-based repo resolution in internal, ingester, git, and router

#1136 knotserver/xrpc: use repo DID resolution in all handlers

#1135 knotserver: add repodid package, db migrations, and go-didplc dep

#1134 lexicons, api/tangled: add repoDid field to all repo-scoped lexicons

ABANDONED PULLS 2

#1141 appview: update repo, pulls, issues, and pages handlers for repo DID

#1139 appview/db: update query functions for repo_did across all tables

Labels

None yet.

assignee

None yet.

Participants 2

AT URI

at://did:plc:3fwecdnvtcscjnrx2p4n7alz/sh.tangled.repo.pull/3mgprvt2eqy22

-6

18 changed files

Interdiff #0 → #1

expand all

appview

issues

issues.go

pages

funcmap.go

repoinfo

repoinfo.go

templates

repo

fork.html

new.html

pulls

pulls.go

repo

archive.go

artifact.go

blob.go

branches.go

compare.go

index.go

log.go

repo.go

settings.go

tags.go

tree.go

webhooks.go

appview/issues/issues.go

This file has not been changed.

appview/pages/funcmap.go

This file has not been changed.

-6

appview/pages/repoinfo/repoinfo.go

··· 20 20 } 21 21 22 22 func (r RepoInfo) FullName() string { 23 - if r.RepoDid != "" { 24 - return r.RepoDid 25 - } 26 23 return path.Join(r.owner(), r.Name) 27 24 } 28 25 ··· 35 32 } 36 33 37 34 func (r RepoInfo) FullNameWithoutAt() string { 38 - if r.RepoDid != "" { 39 - return userutil.FlattenDid(r.RepoDid) 40 - } 41 35 return path.Join(r.ownerWithoutAt(), r.Name) 42 36 } 43 37

appview/pages/templates/repo/fork.html

This file has not been changed.

appview/pages/templates/repo/new.html

This file has not been changed.

appview/pulls/pulls.go

This file has not been changed.

appview/repo/archive.go

This file has not been changed.

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/repo/webhooks.go

This file has not been changed.

History

9 rounds 2 comments

sign up or login to add to the discussion

oyster.cafe submitted #8 6d

1 commit

expand

d5dee7aa

appview: update repo, pulls, issues, and pages handlers for repo DID

expand 2 comments

boltless.me 4d

Uh, in general I want to abandon like half of the stacks. I should've be more clear on this at first place.

Appview doesn't need to know about repo-did situation in most of the cases yet. If repoDid is optional value, there is no reason to use it. Appview still has to support legacy at-uri based connection, so I'd suggest to not introduce extra behavior on referencing and only use repo-did when performing the repository migration. We don't need to touch any tables except the repos itself.

Appview will

ingest repo-did information from sh.tangled.repo records,
store that extra identifier in repos table,
and use that unique identifier when doing redirection/backlinking after owner migration.

No need to change other data.

oyster.cafe 4d

I'll do some cutting then let you know! :p

This pull has been deleted (possibly by jj abandon or jj squash)

oyster.cafe submitted #7 6d

1 commit

expand

d5dee7aa

appview: update repo, pulls, issues, and pages handlers for repo DID

expand 0 comments

oyster.cafe submitted #6 7d

1 commit

expand

d6349c7e

appview: update repo, pulls, issues, and pages handlers for repo DID

expand 0 comments

oyster.cafe submitted #5 7d

1 commit

expand

52f173c6

appview: update repo, pulls, issues, and pages handlers for repo DID

expand 0 comments

oyster.cafe submitted #4 10d

1 commit

expand

e8f69bf9

appview: update repo, pulls, issues, and pages handlers for repo DID

expand 0 comments

oyster.cafe submitted #3 10d

1 commit

expand

5a6108af

appview: update repo, pulls, issues, and pages handlers for repo DID

expand 0 comments

oyster.cafe submitted #2 10d

1 commit

expand

82a027f0

appview: update repo, pulls, issues, and pages handlers for repo DID

expand 0 comments

oyster.cafe submitted #1 13d

diff

1 commit

expand

11874a66

appview: update repo, pulls, issues, and pages handlers for repo DID

expand 0 comments

oyster.cafe submitted #0 13d

diff

1 commit

expand

26aa6f0a

appview: update repo, pulls, issues, and pages handlers for repo DID

expand 0 comments