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

Fix broken link for Threshold signatures documentation #3453

Closed
wants to merge 1 commit into from

Conversation

karangoraniya
Copy link

@karangoraniya karangoraniya commented Sep 11, 2024

Fix broken link for Threshold signatures documentation

This pull request corrects a broken link in the documentation for Threshold signatures by removing an double forward slash ("//") from the link URL.

Changes made:

  • Removed double "/" from the Threshold signatures documentation link
  • The corrected link now properly redirects to the intended documentation page

Before: /docs/current/references/t-ecdsa-how-it-works//
After: /docs/current/references/t-ecdsa-how-it-works

This fix ensures that users can access the correct Threshold signatures documentation without encountering a redirect error.

Contribution Checklist:

@sa-github-api
Copy link

Dear @karangoraniya,

In order to potentially merge your code in this open-source repository and therefore proceed with your contribution, we need to have your approval on DFINITY's CLA.

If you decide to agree with it, please visit this issue and read the instructions there. Once you have signed it, re-trigger the workflow on this PR to see if your code can be merged.

— The DFINITY Foundation

@jessiemongeon1
Copy link
Contributor

@karangoraniya Please accept the CLA so that we can merge this PR. Thanks!

@jessiemongeon1
Copy link
Contributor

This edit has been addressed in another PR: #3492

@karangoraniya
Copy link
Author

@karangoraniya Please accept the CLA so that we can merge this PR. Thanks!

@jessiemongeon1
I have accepted the CLA in the repository. Can you confirm that it is correct so that this issue doesn't happen next time?

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

Successfully merging this pull request may close these issues.

3 participants