tangled.org / core

fork atom

Monorepo for Tangled tangled.org

fork atom

appview: use less `RepoInfo` for `ResolvedRepo` #810

merged opened by

boltless.me 4 months ago targeting master from sl/yurolxtlpsmz

RepoInfo is mostly designed for rendering, don't use it for service logic.

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

STACK 11

#821 appview: remove ResolvedRepo for eternity

3 comments

round #2

#820 appview: remove use of all fields under ResolvedRepo

0 comments

round #2

#812 appview: prefer models.Repo over ResolvedRepo internally

0 comments

round #3

#811 appview/pages: fix pages.RepoComparedDiffFragment()

0 comments

round #4

#810 appview: use less RepoInfo for ResolvedRepo

0 comments

round #4

#809 appview: remove ResolvedRepo.RolesInRepo()

0 comments

round #4

#808 appview: diet the reporesolver

0 comments

round #4

#807 appview: remove unused fields on RepoInfo

0 comments

round #4

#802 appview: use less OwnerSlashRepo() in handlers

1 comment

round #5

#801 appview/pages: resolve did on render

0 comments

round #5

#806 appview/db: remove unused struct

0 comments

round #4

Labels

None yet.

assignee

None yet.

Participants 1

AT URI

at://did:plc:xasnlahkri4ewmbuzly2rlc5/sh.tangled.repo.pull/3m5rjwck3th22

8 changed files

Interdiff #2 → #3

expand all

appview

issues

issues.go

pipelines

pipelines.go

pulls

pulls.go

repo

compare.go

index.go

log.go

repo.go

repo_util.go

appview/issues/issues.go

This file has not been changed.

appview/pipelines/pipelines.go

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

appview/pulls/pulls.go

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

appview/repo/compare.go

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

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/repo_util.go

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

History

5 rounds 0 comments

boltless.me submitted #4 3mo

diff interdiff

1 commit

expand

13f353c4

appview: use less RepoInfo for ResolvedRepo

expand 0 comments

pull request successfully merged

boltless.me submitted #3 3mo

diff

1 commit

expand

13f353c4

appview: use less RepoInfo for ResolvedRepo

expand 0 comments

boltless.me submitted #2 4mo

diff interdiff

1 commit

expand

cbd86f00

appview: use less RepoInfo for ResolvedRepo

expand 0 comments

boltless.me submitted #1 4mo

diff interdiff

1 commit

expand

bd3d57bf

appview: use less RepoInfo for ResolvedRepo

2/3 failed, 1/3 success

expand

expand 0 comments

boltless.me submitted #0 4mo

diff

1 commit

expand

da371810

appview: use less RepoInfo for ResolvedRepo

2/3 failed, 1/3 success

expand

expand 0 comments