Skip to content

added missing hyperlink#1204

Merged
WesHarrell merged 1 commit intomainfrom
fix-mynesi-link
Mar 25, 2026
Merged

added missing hyperlink#1204
WesHarrell merged 1 commit intomainfrom
fix-mynesi-link

Conversation

@WesHarrell
Copy link
Copy Markdown
Collaborator

No description provided.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 25, 2026

Test deployment successful!! (2026-03-25T00:54:57Z)
Preview available at https://callumwalley.github.io/mkdocs-demo-deploy/nesi/support-docs/fix-mynesi-link

Seems the following pages differ;



See all deployed demo sites

Copy link
Copy Markdown
Contributor

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

Choose a reason for hiding this comment

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

Code Review

Ahoy there! This here pull request be a simple one, changing a plain URL to a proper Markdown link in the 'Account Requests for non-Tuakiri Members' document. However, a keen eye has spotted a bit of a blunder, for ye've named the link 'MyNeSI' when elsewhere in these very scrolls, and across the seven seas of this repository, it be known as my.nesi.org.nz. Such inconsistency be a navigational hazard for any poor soul trying to find their way, and it goes against the very spirit of keeping our charts clear, as laid out in our own style guide (lines 483-500). Best to keep the names consistent, lest ye wish to confuse the crew.

an automatically generated email inviting you to activate your account.
You will need to activate your account before you can log in to
my.nesi.org.nz.
[MyNeSI](https://my.nesi.org.nz).
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.

low

Ahoy, matey! While I see ye've added a link, ye've made the map a bit of a muddle. Elsewhere in this very scroll (line 18), and across the seas of this repository, the portal is marked as my.nesi.org.nz. Using MyNeSI here creates an inconsistency that could confuse a poor soul. Let's keep our charts clear and consistent, shall we?

Suggested change
[MyNeSI](https://my.nesi.org.nz).
[my.nesi.org.nz](https://my.nesi.org.nz).
References
  1. The style guide has a section on link clarity (lines 483-500). While not a direct violation, maintaining consistency in how links are presented is crucial for readability and to avoid confusion, which is in the spirit of the guide. This change introduces an inconsistent link style within the same document. (link)

@WesHarrell WesHarrell merged commit 92bc6a4 into main Mar 25, 2026
8 checks passed
@WesHarrell WesHarrell deleted the fix-mynesi-link branch March 25, 2026 00:55
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.

1 participant