Skip to content

fix: Corrected fido-2-devices link to heading in user management document. Issue #9636 #10794

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

Merged

Conversation

DanielSLDev
Copy link
Contributor

Prepare


Description

Link to fido-2-devices was not working correctly due to missing "-" in the link. It is now added and it is now working.

Target issue

#9636

closes #9636


Test and Document the changes

  • Static code analysis has been run locally and issues have been fixed
  • Relevant unit and integration tests have been added/updated
  • Relevant documentation has been updated if any (i.e. user guides, installation and configuration guides, technical design docs etc)

Please check the below before submitting your PR. The PR will not be merged if there are no commits that start with docs: to indicate documentation changes or if the below checklist is not selected.

  • I confirm that there is no impact on the docs due to the code changes in this PR.

…ment. issue#9636.

Signed-off-by: Daniel Salas <daniel.salas@encora.com>
@DanielSLDev
Copy link
Contributor Author

Hi @ossdhaval. The signatures have been added, and verified. The issue has been fixed. Please let me know if there is any other feedback for my contribution.:)

@DanielSLDev
Copy link
Contributor Author

@ossdhaval friendly reminder.

@ossdhaval ossdhaval enabled auto-merge (squash) February 7, 2025 13:20
@ossdhaval
Copy link
Contributor

Submission looks perfect this time 🎉

Reviewed and approved. 👍

@ossdhaval ossdhaval merged commit 8ffc483 into JanssenProject:main Feb 7, 2025
2 checks passed
@DanielSLDev DanielSLDev deleted the jannsen-server-fixWrongLink branch February 11, 2025 17:52
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 this pull request may close these issues.

fix(docs): incorrect link to heading in user management document
3 participants