Skip to content

chore: fix some typos in comment #821

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 1 commit into from

Conversation

toofooboo
Copy link
Contributor

@toofooboo toofooboo commented Dec 20, 2024

Changes

fix some typos in comment

How this PR will be tested

  • Open the dashboard and click X. Result: A modal should appear.
  • Call the /foo/bar API. Result: Returns 200 with "baz" in the response body.

Output

(Example: Screenshot/GIF for UI changes, cURL output for API changes)


PR-Codex overview

This PR focuses on correcting typos in comments throughout the codebase, specifically in the send-transaction-queue.ts, import.ts, queue-transaction.ts, and create-wallet-details.ts files.

Detailed summary

  • Corrected "enqueing" to "enqueuing" in send-transaction-queue.ts.
  • Fixed "endoint" to "endpoint" in import.ts.
  • Changed "depcrecated" to "deprecated" in create-wallet-details.ts for several comments.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

Signed-off-by: toofooboo <cmaker@foxmail.com>
Copy link

This PR is stale because it has been open for 7 days with no activity. Remove stale label or comment or this PR will be closed in 3 days.

@toofooboo
Copy link
Contributor Author

friendly ping~

@arcoraven arcoraven enabled auto-merge (squash) December 31, 2024 07:23
@arcoraven
Copy link
Contributor

Thanks for the contribution. This repo requires signed commits. Can you set up your github to sign commits? https://docs.github.com/en/authentication/managing-commit-signature-verification/signing-commits

Alternatively I can clone this PR and push the changes up: #822

@arcoraven arcoraven closed this Dec 31, 2024
auto-merge was automatically disabled December 31, 2024 08:34

Pull request was closed

@arcoraven
Copy link
Contributor

Closing and merged with #822

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