tangled.org
The compare forks option dropdown now shows repos in which you are a collaborator on instead of just repos you own.
Monorepo for Tangled — https://tangled.org
appview: Update New PR Page #490
merged
opened by
freshlybakedca.ke
targeting
master
from
freshlybakedca.ke/core:
private/coded/push-nvwonuuosssx
expand 2 commits
hide 2 commits
appview: db: Update GetForksByDid
Signed-off-by: Samuel Shuert <me@thecoded.prof>
appview: pages: Update PRCompare fork titles
Show fork titles in owner/repo format, otherwise if you have collaborator
access to multiple forks of the same repo (in which the forks have the
same title) it may be hard to differentiate
Signed-off-by: Samuel Shuert <me@thecoded.prof>
Updated the underlying logic for both that function and
func (s *Pulls) CompareForksBranchesFragment(w http.ResponseWriter, r *http.Request)
which was the other one we talked about on discord
expand 2 commits
hide 2 commits
appview: db: Update GetForksByDid
Signed-off-by: Samuel Shuert <me@thecoded.prof>
appview: pages: Update PRCompare fork page
Show fork titles in owner/repo format, otherwise if you have collaborator
access to multiple forks of the same repo (in which the forks have the
same title) it may be hard to differentiate
Also update the underlying logic to use the option value instead of
checking against the currently logged in users DID
Signed-off-by: Samuel Shuert <me@thecoded.prof>
expand 2 commits
hide 2 commits
appview: db: update GetForksByDid function
Signed-off-by: Samuel Shuert <me@thecoded.prof>
appview: pages: update pr compare fork page
Show fork titles in owner/repo format, otherwise if you have collaborator
access to multiple forks of the same repo (in which the forks have the
same title) it may be hard to differentiate
Also update the underlying logic to use the option value instead of
checking against the currently logged in users DID
Signed-off-by: Samuel Shuert <me@thecoded.prof>
expand 2 commits
hide 2 commits
appview: db: update GetForksByDid function
Signed-off-by: Samuel Shuert <me@thecoded.prof>
appview: pages: update pr compare fork page
Show fork titles in owner/repo format, otherwise if you have collaborator
access to multiple forks of the same repo (in which the forks have the
same title) it may be hard to differentiate
Also update the underlying logic to use the option value instead of
checking against the currently logged in users DID
Signed-off-by: Samuel Shuert <me@thecoded.prof>
expand 2 commits
hide 2 commits
appview: db: update GetForksByDid function
Signed-off-by: Samuel Shuert <me@thecoded.prof>
appview: pages: update pr compare fork page
Show fork titles in owner/repo format, otherwise if you have collaborator
access to multiple forks of the same repo (in which the forks have the
same title) it may be hard to differentiate
Also update the underlying logic to use the option value instead of
checking against the currently logged in users DID
Signed-off-by: Samuel Shuert <me@thecoded.prof>
expand 2 commits
hide 2 commits
appview: db: update GetForksByDid function
Signed-off-by: Samuel Shuert <me@thecoded.prof>
appview: pages: update pr compare fork page
Show fork titles in owner/repo format, otherwise if you have collaborator
access to multiple forks of the same repo (in which the forks have the
same title) it may be hard to differentiate
Also update the underlying logic to use the option value instead of
checking against the currently logged in users DID
Signed-off-by: Samuel Shuert <me@thecoded.prof>
expand 2 commits
hide 2 commits
appview: db: update GetForksByDid function
Co-Authored-By: oppiliappan <me@oppi.li>
Signed-off-by: Samuel Shuert <me@thecoded.prof>
appview: pages: update pr compare fork page
Show fork titles in owner/repo format, otherwise if you have collaborator
access to multiple forks of the same repo (in which the forks have the
same title) it may be hard to differentiate
Also update the underlying logic to use the option value instead of
checking against the currently logged in users DID
Co-Authored-By: oppiliappan <me@oppi.li>
Signed-off-by: Samuel Shuert <me@thecoded.prof>
expand 2 commits
hide 2 commits
appview: db: update GetForksByDid function
Co-Authored-By: oppiliappan <me@oppi.li>
Signed-off-by: Samuel Shuert <me@thecoded.prof>
appview: pages: update pr compare fork page
Show fork titles in owner/repo format, otherwise if you have collaborator
access to multiple forks of the same repo (in which the forks have the
same title) it may be hard to differentiate
Also update the underlying logic to use the option value instead of
checking against the currently logged in users DID
Co-Authored-By: oppiliappan <me@oppi.li>
Signed-off-by: Samuel Shuert <me@thecoded.prof>
expand 2 commits
hide 2 commits
appview: db: update GetForksByDid function
Co-Authored-By: oppiliappan <me@oppi.li>
Signed-off-by: Samuel Shuert <me@thecoded.prof>
appview: pages: update pr compare fork page
Show fork titles in owner/repo format, otherwise if you have collaborator
access to multiple forks of the same repo (in which the forks have the
same title) it may be hard to differentiate
Also update the underlying logic to use the option value instead of
checking against the currently logged in users DID
Co-Authored-By: oppiliappan <me@oppi.li>
Signed-off-by: Samuel Shuert <me@thecoded.prof>
copying over my comment from discord just for my own reference:
we will need some changes to the backend also here (in pulls/pulls.go):
the form that you have edited has a select where the
options value is just the repo name, we then use the logged in user's DID to figure out the fork to use, so we constructuserDId/repoName(whereas this DID should also come from the form)