-
Notifications
You must be signed in to change notification settings - Fork 154
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Broken links to remote repositories #1689
Comments
Report links with remote repositories that are supported and not GitHub (either GitLab or Bitbucket) are expected to be fixed in #1644. |
Current links behaviour on master branch as of 1 April 2022 #1644 has standardized the behaviour of links to be
#1679 should allow partial support of local repos cloned from remote repos to support links that are already present on the remote repo. |
Besides the two suggestions you have, I might add an 'intermediate' suggestion where
|
@chan-j-d Actually I prefer the button to be hidden entirely or disabled because sometimes we click the button before seeing the hover message. |
I think I agree. There is no ambiguity that a link is not supported if there isn't a link (icon) in the first place |
Hi is this issue still open? It is OK to work on this issue? |
Sure, I've assigned you the issue! |
OK thanks! Shall I hide the button when the link doesn't work? |
Yes, that appears to be the conclusion from above. @Zhou-Jiahao-1998, do update this thread if you have better ideas for handling missing/unsupported remotes. |
@gok99 Actually now that I think about it again, I prefer disabling the button and add a hover message. That way the user won't be able to click and is likely gonna see an explanation in the hover message. Hiding without stating anything might cause confusion? |
@Zhou-Jiahao-1998 That makes sense! @LuoZhijie-tom You could do this instead. Perhaps something like |
OK sure! |
There are some links in the report to the report repos such as view group repo. However, when the link is missing or not supported by RepoSense (i.e. is not Github, GitLab or Bitbucket), it points to a non-existent link. Let's disable the link when it is broken and change the message when the users hover over the link to "This remote link is unsupported", so that the users will know that the link is broken.
This issue consolidates issues with links in the report to remote repositories. These links should be fixed for when (a) no remote repositories exist or (b) remote repository is not yet supported by RepoSense (is not Github, GitLab or Bitbucket). The following report links redirect to broken links if either of the above conditions are satisfied.
To discuss:
What should be the expected behaviour in the above scenarios if either (a) or (b) is satisfied? Some options:
The text was updated successfully, but these errors were encountered: