The compare forks option dropdown now shows repos in which you are a collaborator on instead of just repos you own.
+8
-5
appview/db/repos.go
+8
-5
appview/db/repos.go
···
466
var repos []Repo
467
468
rows, err := e.Query(
469
+
`select distinct r.did, r.name, r.knot, r.rkey, r.description, r.created, r.source
470
+
from repos r
471
+
left join collaborators c on r.at_uri = c.repo_at
472
+
where (r.did = ? or c.subject_did = ?)
473
+
and r.source is not null
474
+
and r.source != ''
475
+
order by r.created desc`,
476
+
did, did,
477
)
478
if err != nil {
479
return nil, err
+2
-2
appview/pages/templates/repo/pulls/fragments/pullCompareForks.html
+2
-2
appview/pages/templates/repo/pulls/fragments/pullCompareForks.html
+11
-6
appview/pulls/pulls.go
+11
-6
appview/pulls/pulls.go
···
857
}
858
859
func (s *Pulls) handleForkBasedPull(w http.ResponseWriter, r *http.Request, f *reporesolver.ResolvedRepo, user *oauth.User, forkRepo string, title, body, targetBranch, sourceBranch string, isStacked bool) {
860
-
fork, err := db.GetForkByDid(s.db, user.Did, forkRepo)
861
if errors.Is(err, sql.ErrNoRows) {
862
s.pages.Notice(w, "pull", "No such fork.")
863
return
···
888
return
889
}
890
891
-
resp, err := sc.NewHiddenRef(user.Did, fork.Name, sourceBranch, targetBranch)
892
if err != nil {
893
log.Println("failed to create hidden ref:", err, resp.StatusCode)
894
s.pages.Notice(w, "pull", "Failed to create pull request. Try again later.")
···
908
// hiddenRef: hidden/feature-1/main (on repo-fork)
909
// targetBranch: main (on repo-1)
910
// sourceBranch: feature-1 (on repo-fork)
911
-
comparison, err := us.Compare(user.Did, fork.Name, hiddenRef, sourceBranch)
912
if err != nil {
913
log.Println("failed to compare across branches", err)
914
s.pages.Notice(w, "pull", err.Error())
···
1270
}
1271
1272
forkVal := r.URL.Query().Get("fork")
1273
-
1274
// fork repo
1275
-
repo, err := db.GetRepo(s.db, user.Did, forkVal)
1276
if err != nil {
1277
log.Println("failed to get repo", user.Did, forkVal)
1278
return
···
1285
return
1286
}
1287
1288
-
sourceResult, err := sourceBranchesClient.Branches(user.Did, repo.Name)
1289
if err != nil {
1290
log.Println("failed to reach knotserver for source branches", err)
1291
return
···
857
}
858
859
func (s *Pulls) handleForkBasedPull(w http.ResponseWriter, r *http.Request, f *reporesolver.ResolvedRepo, user *oauth.User, forkRepo string, title, body, targetBranch, sourceBranch string, isStacked bool) {
860
+
repoString := strings.SplitN(forkRepo, "/", 2)
861
+
forkOwnerDid := repoString[0]
862
+
repoName := repoString[1]
863
+
fork, err := db.GetForkByDid(s.db, forkOwnerDid, repoName)
864
if errors.Is(err, sql.ErrNoRows) {
865
s.pages.Notice(w, "pull", "No such fork.")
866
return
···
891
return
892
}
893
894
+
resp, err := sc.NewHiddenRef(fork.Did, fork.Name, sourceBranch, targetBranch)
895
if err != nil {
896
log.Println("failed to create hidden ref:", err, resp.StatusCode)
897
s.pages.Notice(w, "pull", "Failed to create pull request. Try again later.")
···
911
// hiddenRef: hidden/feature-1/main (on repo-fork)
912
// targetBranch: main (on repo-1)
913
// sourceBranch: feature-1 (on repo-fork)
914
+
comparison, err := us.Compare(fork.Did, fork.Name, hiddenRef, sourceBranch)
915
if err != nil {
916
log.Println("failed to compare across branches", err)
917
s.pages.Notice(w, "pull", err.Error())
···
1273
}
1274
1275
forkVal := r.URL.Query().Get("fork")
1276
+
repoString := strings.SplitN(forkVal, "/", 2)
1277
+
forkOwnerDid := repoString[0]
1278
+
forkName := repoString[1]
1279
// fork repo
1280
+
repo, err := db.GetRepo(s.db, forkOwnerDid, forkName)
1281
if err != nil {
1282
log.Println("failed to get repo", user.Did, forkVal)
1283
return
···
1290
return
1291
}
1292
1293
+
sourceResult, err := sourceBranchesClient.Branches(forkOwnerDid, repo.Name)
1294
if err != nil {
1295
log.Println("failed to reach knotserver for source branches", err)
1296
return