-
-
Notifications
You must be signed in to change notification settings - Fork 766
Update: Implementations Links added in Aliquot_Sum.md #216
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
Update: Implementations Links added in Aliquot_Sum.md #216
Conversation
Added Implementations Links in Aliquot_Sum.md
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please replace the specific implementation links with one link to the TheAlgorithms site for the Aliquot sum. See #190.
Are you mean to say, i need to remove all implementation link and add only one link to the TheAlgorithms site for the Aliquot sum. For ex: Then what about the below links in website it self? It will be replace by one link only after i'll replace to one link right? so it will show the same URL of the website when user will click on this implementation link. Please confirm. What should i do! Thanks. |
Yes.
Indeed it will be a link to the same page the user is already on. This is a redundancy problem: The links are (presumably) there for when the explanations are viewed on GitHub, but on the website they are redundant. This is a website issue IMO. |
Added Single Implementation Link of `The Algorithms` website in Aliquot_Sum.md
@appgurueu As per your suggestion, I added single link to the TheAlgorithms site for the Aliquot sum. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Though you could omit the -Aliquot-Sum
@appgurueu Yeah, Sure i have omitted |
Hi @vbrazo and @Panquesito7 Please review this PR. Thanks. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you! 🚀
Added Implementations Links in Aliquot_Sum.md
Description
Closes #{ISSUE_ID}
Checklist
Screenshots (if any)
Note to reviewers