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

README src/calculateRank.js Link Inconsistent #3066

Open
conorvenus opened this issue Aug 13, 2023 · 10 comments · Fixed by #3067 or #3389
Open

README src/calculateRank.js Link Inconsistent #3066

conorvenus opened this issue Aug 13, 2023 · 10 comments · Fixed by #3067 or #3389
Labels
bug Something isn't working.

Comments

@conorvenus
Copy link

Describe the bug

The src/calculateRank.js link inside the README file does not work consistently, i.e., if you are on https://github.com/anuraghazra/github-readme-stats and you click it, it takes you to https://github.com/anuraghazra/src/calculateRank.js incorrectly.

Expected behavior

I would expect to be taken to the src/calculateRank.js file within the current repository.

Screenshots / Live demo link

https://streamable.com/ocjjn1

Additional context

No response

@conorvenus conorvenus added the bug Something isn't working. label Aug 13, 2023
@rickstaa
Copy link
Collaborator

Thanks for the bug report. This will be fixed when #3067 is merged 👍🏻.

a-s-t-e-y-a added a commit to a-s-t-e-y-a/github-readme-stats that referenced this issue Oct 1, 2023
a-s-t-e-y-a added a commit to a-s-t-e-y-a/github-readme-stats that referenced this issue Oct 1, 2023
a-s-t-e-y-a added a commit to a-s-t-e-y-a/github-readme-stats that referenced this issue Oct 1, 2023
@a-s-t-e-y-a
Copy link

@rickstaa I solved the readme.md issue by fixing the link to https://github.com/anuraghazra/github-readme-stats/blob/master/src/calculateRank.js

@rickstaa
Copy link
Collaborator

Looks like there is a bug on GitHub's side. I just opened a ticket with the support.

image

Originally posted in #3140.

@qwerty541
Copy link
Collaborator

Looks like there is a bug on GitHub's side. I just opened a ticket with the support.

image

Originally posted in #3140.

Nice, let's wait for their response.

@rickstaa
Copy link
Collaborator

Looks like there is a bug on GitHub's side. I just opened a ticket with the support.
image

Originally posted in #3140.

Nice, let's wait for their response.

I just got several responses:

image

image

It looks like a bug in their system, which they are trying to solve👍🏻.

@qwerty541
Copy link
Collaborator

Looks like there is a bug on GitHub's side. I just opened a ticket with the support.
image

Originally posted in #3140.

Nice, let's wait for their response.

I just got several responses:

image

image

It looks like a bug in their system, which they are trying to solve👍🏻.

@rickstaa Should we use absolute path until this bug will be fixed in your opinion? It can take some weeks or months untils patched version will be deployed on production.

@rickstaa
Copy link
Collaborator

Looks like there is a bug on GitHub's side. I just opened a ticket with the support.
image

Originally posted in #3140.

Nice, let's wait for their response.

I just got several responses:
image
image
It looks like a bug in their system, which they are trying to solve👍🏻.

@rickstaa Should we use absolute path until this bug will be fixed in your opinion? It can take some weeks or months untils patched version will be deployed on production.

I'm okay with merging a pull request that uses the absolute path, keeping this issue open, and reverting it later when the problem is fixed on GitHub's side 👍🏻.

@qwerty541
Copy link
Collaborator

Looks like there is a bug on GitHub's side. I just opened a ticket with the support.
image

Originally posted in #3140.

Nice, let's wait for their response.

I just got several responses:
image
image
It looks like a bug in their system, which they are trying to solve👍🏻.

@rickstaa Should we use absolute path until this bug will be fixed in your opinion? It can take some weeks or months untils patched version will be deployed on production.

I'm okay with merging a pull request that uses the absolute path, keeping this issue open, and reverting it later when the problem is fixed on GitHub's side 👍🏻.

I have opened a pull request with temporary solution for both src/calculateRank.js and vercel.json files. Please check it #3389

@rickstaa
Copy link
Collaborator

rickstaa commented Oct 18, 2023

Looks like there is a bug on GitHub's side. I just opened a ticket with the support.
image

Originally posted in #3140.

Nice, let's wait for their response.

I just got several responses:
image
image
It looks like a bug in their system, which they are trying to solve👍🏻.

@rickstaa Should we use absolute path until this bug will be fixed in your opinion? It can take some weeks or months untils patched version will be deployed on production.

I'm okay with merging a pull request that uses the absolute path, keeping this issue open, and reverting it later when the problem is fixed on GitHub's side 👍🏻.

I have opened a pull request with temporary solution for both src/calculateRank.js and vercel.json files. Please check it #3389

Great, let's keep this issue open as a reference! I will inform you when GitHub emails me that the bug is fixed 👍🏻.

@qwerty541
Copy link
Collaborator

Looks like there is a bug on GitHub's side. I just opened a ticket with the support.
image

Originally posted in #3140.

Nice, let's wait for their response.

I just got several responses:
image
image
It looks like a bug in their system, which they are trying to solve👍🏻.

@rickstaa Should we use absolute path until this bug will be fixed in your opinion? It can take some weeks or months untils patched version will be deployed on production.

I'm okay with merging a pull request that uses the absolute path, keeping this issue open, and reverting it later when the problem is fixed on GitHub's side 👍🏻.

I have opened a pull request with temporary solution for both src/calculateRank.js and vercel.json files. Please check it #3389

Great, let's keep this issue open as a reference! I will inform you when GitHub emails me that the bug is fixed 👍🏻.

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working.
Projects
None yet
4 participants