Skip to content

add troubleshooting section for trusted device naming issues#1316

Merged
WesHarrell merged 1 commit into
mainfrom
trusted-device-error
Jun 23, 2026
Merged

add troubleshooting section for trusted device naming issues#1316
WesHarrell merged 1 commit into
mainfrom
trusted-device-error

Conversation

@WesHarrell

Copy link
Copy Markdown
Collaborator

closes 1053

@WesHarrell WesHarrell requested a review from natnesi June 23, 2026 02:57
@github-actions

github-actions Bot commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

Test deployment successful!! (2026-06-23T02:57:41Z)
Preview available at https://callumwalley.github.io/mkdocs-demo-deploy/nesi/support-docs/trusted-device-error

Seems the following pages differ;



See all deployed demo sites

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

Ahoy! This patch adds a new section to the login troubleshooting guide for when trusting a device results in an error. But shiver me timbers, ye made some rookie blunders! Ye left the image alt text completely empty, leaving blind sailors stranded in the dark, and ye used a fancy H4 header to style a simple warning instead of using a proper warning admonition. Fix these, ye scallywag, or prepare to walk the plank!

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.


## Trusting Device results in error

![](../../assets/images/Trusted-Device-error.png)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

Ahoy! Ye left the alt text completely empty! How be a poor blind sailor supposed to know what treasure lies in this image? Or did ye think we all have magical spyglasses that can see through blank brackets? Add some descriptive alt text, ye scallywag!

Suggested change
![](../../assets/images/Trusted-Device-error.png)
![Trusted Device Error Dialog](../../assets/images/Trusted-Device-error.png)
References
  1. Images should have descriptive alt text for accessibility standards (WCAG/NZ spec). (link)

Comment on lines +78 to +82


#### This error is caused by a naming conflict, in other words you have used the same name previously. Choose a different device name and try again


Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

Shiver me timbers! Why are ye usin' an H4 header (####) for a mere sentence? Did ye think headers are just fancy decorations to make yer text look pretty? Use a proper warning admonition instead, unless ye want to face the captain's wrath and walk the plank!

Suggested change
#### This error is caused by a naming conflict, in other words you have used the same name previously. Choose a different device name and try again
!!! warning "Naming Conflict"
This error is caused by a naming conflict, meaning you have used the same name previously. Choose a different device name and try again.
References
  1. Do not use headers for styling regular text. Use admonitions for warnings or callouts. (link)

@WesHarrell

Copy link
Copy Markdown
Collaborator Author

oops, forgot the hash, closes #1053

@natnesi natnesi left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thank you @WesHarrell , it is exactly that. thank you!

@WesHarrell WesHarrell merged commit 871d7ad into main Jun 23, 2026
8 checks passed
@WesHarrell WesHarrell deleted the trusted-device-error branch June 23, 2026 03:13
@WesHarrell WesHarrell linked an issue Jun 23, 2026 that may be closed by this pull request
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.

Add keycloack trusted device error

2 participants