Skip to content

fix: docs fixups - #18

Draft
leohhhn wants to merge 5 commits into
mainfrom
feat/text-modifications
Draft

leohhhn wants to merge 5 commits into
mainfrom
feat/text-modifications

Conversation

@leohhhn

@leohhhn leohhhn commented Oct 29, 2026 •

Copy link
Copy Markdown

Description

Fixups and things I'm finding along the way while reading the docs.

@leohhhn

leohhhn commented Oct 29, 2026

Copy link
Copy Markdown
Author

need to check the formatting style of the repo

@leohhhn leohhhn self-assigned this Oct 29, 2026
@leohhhn
leohhhn marked this pull request as draft October 29, 2026 18:36
@leohhhn

leohhhn commented Oct 29, 2026

Copy link
Copy Markdown
Author

Should we consider adding a .editorconfig or .prettierrc?

@leohhhn
leohhhn marked this pull request as ready for review October 29, 2026 18:44
@leohhhn
leohhhn requested review from HAPPYS1NGH and thecaphimself and removed request for HAPPYS1NGH October 29, 2026 18:44
@leohhhn leohhhn changed the title fix: formalize text fix: docs fixups Oct 29, 2026
Comment thread user-guide/app.mdx
<Step title="Open the Onchain section">
Visit [app.namespace.ninja/onchain](https://app.namespace.ninja/onchain), connect the owner wallet, select the ENS name, and click **Activate Name**.

<img src="/assets/activate-ens-name-1.png" alt="Onchain section with an ENS name selected and the Activate Name button" />

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

These screens are a bit outdated as far as I can see, minor but could be updated

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Playwright might be overkill


See these live examples of customized Subpages:

- **OP Punks**: [oppunk.namespace.ninja](http://oppunk.namespace.ninja)

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

This one is dead - not sure if that should be the case. Maybe we should have a url checker?

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

@nenadmitt - wanna check.

@leohhhn

leohhhn commented Oct 29, 2026 •

Copy link
Copy Markdown
Author

Generally speaking, for someone who is learning about how ENS works on the technical side, I'd like some more info on how off-chain subname resolution works, or at least some pointers to ENS docs but the info is scattered there as well.

Possible technical content for docs/X:

  • detailed resolution diagram - explaining why using CCIP reads matters, how request flow, etc. There is a diagram in the "Universal usernames" page but it doesn't provide enough info for someone who doesn't know the details.

@leohhhn

leohhhn commented Oct 29, 2026

Copy link
Copy Markdown
Author

still need to go through the developer guide section - tomorrow

@leohhhn
leohhhn marked this pull request as draft October 29, 2026 19:49

This branch has not been deployed

No deployments
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.

2 participants