Skip to content

fix(create-form): describe state purpose directly (FlowwStar/FlowStar#385)#388

Open
jdjioe5-cpu wants to merge 1 commit into
FlowwStar:mainfrom
jdjioe5-cpu:fix/issue-385-comment-rewrite
Open

fix(create-form): describe state purpose directly (FlowwStar/FlowStar#385)#388
jdjioe5-cpu wants to merge 1 commit into
FlowwStar:mainfrom
jdjioe5-cpu:fix/issue-385-comment-rewrite

Conversation

@jdjioe5-cpu

Copy link
Copy Markdown

Summary

Closes #385.

Change

Replaces two stale issue-number comments with descriptive comments:

  • L158: // Issue #29: balance state// Tracks the connected wallet's token balance used during stream-creation validation.
  • L162: // Issue #103: recipient account validation// Tracks whether the recipient account exists and is funded before allowing a stream to be created.

No logic change. git diff shows 2 insertions, 2 deletions.

Replaces two // Issue #XX comments in create-form.tsx that referenced
resolved GitHub issues with comments that describe what the code does:

- L158: token-balance state use during stream-creation validation
- L162: recipient-account validation state (existence + funded)

Closes FlowwStar#385.
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.

good first issue: fix stale GitHub issue-number comments in create-form.tsx

1 participant