Skip to content
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

support for self-hosting gitlab-ce version (path unusing - dash) #37

Closed
norimsu opened this issue Jun 1, 2020 · 8 comments · Fixed by #40
Closed

support for self-hosting gitlab-ce version (path unusing - dash) #37

norimsu opened this issue Jun 1, 2020 · 8 comments · Fixed by #40

Comments

@norimsu
Copy link

norimsu commented Jun 1, 2020

Hi, I'm happy to find this plug-in.

But there is some problem with using it.

I use self-hosting gitlab-ce.

The link requested by "/~/blob/..." does not work.

The other addresses work well except for "/~".

http://some-gitlab.com/some-group/some-project/~/blob/some-file.code   # doesn't work - 404
http://some-gitlab.com/some-group/some-project/blob/some-file.code     # work

The GitLab-CE version I use is '12.3.4 (f33961a395)'

What do you think about adding this content as an option?

@norimsu norimsu changed the title support for self-hosting gitlab-ce version (path using ~ tilt) support for self-hosting gitlab-ce version (path unusing ~ tilt) Jun 1, 2020
@tavyandy97
Copy link
Owner

tavyandy97 commented Jun 1, 2020

window.location.href = `${window.location.origin}/${
URLDetails.dirFormatted
}/-/blob/${URLDetails.branchName}/${path.join("/")}`;

None of the redirections in SpanTree involve the use of a ~(tilde).

1. http://some-gitlab.com/some-group/some-project/~/blob/some-file.code    # doesn't work - 404
2. http://some-gitlab.com/some-group/some-project/-/blob/some-file.code    # works
3. http://some-gitlab.com/some-group/some-project/blob/some-file.code      # works

Can you tell me when you click on a file in SpanTree, which link appears from the above?
Just for reference SpanTree uses 2 for redirection.

@norimsu
Copy link
Author

norimsu commented Jun 1, 2020

oh, it's mistake

- not ~

for GitLab-CE 12.3.4 (f33961a395)

2. http://some-gitlab.com/some-group/some-project/-/blob/some-branch/some-file.code    # does not work - 404
3. http://some-gitlab.com/some-group/some-project/blob/some-branch/some-file.code      # works
4. http://some-gitlab.com/some-group/some-project/tree/some-branch/some-file.code      # works

@norimsu norimsu changed the title support for self-hosting gitlab-ce version (path unusing ~ tilt) support for self-hosting gitlab-ce version (path unusing - tilt) Jun 1, 2020
@norimsu norimsu changed the title support for self-hosting gitlab-ce version (path unusing - tilt) support for self-hosting gitlab-ce version (path unusing - dash) Jun 1, 2020
@tavyandy97
Copy link
Owner

tavyandy97 commented Jun 2, 2020

This is an issue that exists on GitLab (versions < 13).
Thanks for clarifying @norimsu . We have started working on a compatibility mode. It will be released within a week.

@tavyandy97 tavyandy97 mentioned this issue Jun 6, 2020
@vedanshgarg vedanshgarg linked a pull request Jun 7, 2020 that will close this issue
@lifesign
Copy link

same problem here, i installed from Chrome Web Store which version is 0.0.2.3, when will upgrade the store version

@tavyandy97
Copy link
Owner

image

Hey @norimsu and @lifesign

A compatibility mode has been added for the same. This will be updated in v0.0.2.5. It has been submitted for review on the Chrome Webstore(Should be approved within a day). Will be submitted for Mozilla Firefox soon.

@lifesign
Copy link

it works 👍 thx

@tavyandy97
Copy link
Owner

@lifesign, @norimsu
The extension has been updated on the Chrome Webstore to v0.0.2.5. It will updated on Mozilla Firefox soon. I am closing this issue for now.

@norimsu
Copy link
Author

norimsu commented Jun 11, 2020

thx. 👍👍👍👍👍👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants