Skip to content

Fix PaymentMethodMessagingElement forwardRef warning#2436

Merged
tjclawson-stripe merged 5 commits into
stripe:masterfrom
cyxmon:fix-payment-method-messaging-forward-ref
May 18, 2026
Merged

Fix PaymentMethodMessagingElement forwardRef warning#2436
tjclawson-stripe merged 5 commits into
stripe:masterfrom
cyxmon:fix-payment-method-messaging-forward-ref

Conversation

@cyxmon
Copy link
Copy Markdown
Contributor

@cyxmon cyxmon commented May 1, 2026

Summary

  • Remove forwardRef from PaymentMethodMessagingElement because the component only uses an internal native view ref.
  • Avoid React dev warnings from forwardRef receiving a render function with only one parameter.

Test plan

  • yarn prettier --write src/components/PaymentMethodMessagingElement.tsx
  • yarn typescript
  • yarn lint --quiet src/components/PaymentMethodMessagingElement.tsx

Made with Cursor

Co-authored-by: Cursor <cursoragent@cursor.com>
@cyxmon cyxmon requested review from a team as code owners May 1, 2026 23:17
@cla-assistant
Copy link
Copy Markdown

cla-assistant Bot commented May 1, 2026

CLA assistant check
All committers have signed the CLA.

@tjclawson-stripe tjclawson-stripe self-requested a review May 11, 2026 17:34
@tjclawson-stripe
Copy link
Copy Markdown
Collaborator

Hey @cyxmon, thanks for this PR! Can you run yarn api-extractor:update to update the public API file?

tjclawson-stripe and others added 2 commits May 11, 2026 15:45
…hange

Run yarn api-extractor:update so etc/stripe-react-native.api.md matches the
removed forwardRef export shape.

Co-authored-by: Cursor <cursoragent@cursor.com>
@cyxmon
Copy link
Copy Markdown
Contributor Author

cyxmon commented May 12, 2026

Thanks for the callout. I ran yarn api-extractor:update and committed the resulting public API surface update.

@tjclawson-stripe tjclawson-stripe self-requested a review May 14, 2026 20:47
@tjclawson-stripe tjclawson-stripe merged commit 9a51036 into stripe:master May 18, 2026
6 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