Skip to content

docs: clarify interceptor argument examples#11618

Open
TheWaste11202 wants to merge 1 commit into
loopbackio:masterfrom
TheWaste11202:codex/clarify-interceptor-docs
Open

docs: clarify interceptor argument examples#11618
TheWaste11202 wants to merge 1 commit into
loopbackio:masterfrom
TheWaste11202:codex/clarify-interceptor-docs

Conversation

@TheWaste11202
Copy link
Copy Markdown

Checklist

  • DCO (Developer Certificate of Origin) signed in all commits
  • npm test passes on your machine
  • New tests added or existing tests modified to cover all changes
  • Code conforms with the style guide
  • API Documentation in code was updated
  • Documentation in /docs/site was updated
  • Affected artifact templates in packages/cli were updated
  • Affected example projects in examples/* were updated

👉 Check out how to submit a PR 👈

Signed-off-by: z1551778201-ctrl <Nurtai1120@outlook.com>
@TheWaste11202 TheWaste11202 force-pushed the codex/clarify-interceptor-docs branch from f08bdb0 to 24c24a3 Compare May 12, 2026 12:21
Copy link
Copy Markdown
Member

@dhmlau dhmlau left a comment

Choose a reason for hiding this comment

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

@TheWaste11202, thanks for the PR. Your changes LGTM. Just one minor comment about the extra white spaces.

Comment thread docs/site/Interceptor.md
`Name '${name}' is not on the list of '${this.validNames}`,
);
}
const name = invocationCtx.args[0];
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I believe the linter won't pass due to the extra white space

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