Skip to content

Fix typo: this methods -> this method in approveOrder JSDoc#1887

Merged
ryanio merged 1 commit intoProjectOpenSea:mainfrom
Zentex1337:fix/typo-this-methods-allows
Feb 11, 2026
Merged

Fix typo: this methods -> this method in approveOrder JSDoc#1887
ryanio merged 1 commit intoProjectOpenSea:mainfrom
Zentex1337:fix/typo-this-methods-allows

Conversation

@Zentex1337
Copy link
Contributor

Motivation

The JSDoc comment for the approveOrder method contained a grammatical error in both sdk.ts and sdk/fulfillment.ts.

Solution

Changed this methods allows to this method allows in the approveOrder method documentation.

Fixed grammatical error in the approveOrder method documentation
in both sdk.ts and sdk/fulfillment.ts.
Copy link
Collaborator

@ryanio ryanio left a comment

Choose a reason for hiding this comment

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

LGTM — fixes the subject-verb disagreement in both locations. Thanks!

Minor note: these three typo/grammar PRs (#1885, #1886, #1887) could have been combined into a single PR to reduce review overhead. Something to consider for future contributions.

@ryanio ryanio merged commit 60cb8b9 into ProjectOpenSea:main Feb 11, 2026
2 checks passed
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