Skip to content

Conversation

@tonidero
Copy link
Contributor

TODO

@tonidero tonidero added the pr:other A code change that improves performance label Nov 13, 2025
Copy link
Member

@ajpallares ajpallares left a comment

Choose a reason for hiding this comment

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

Makes sense! Some small comments

presentedOfferingContext?: Record<string, unknown>,
_displayCloseButton?: boolean,
_fontFamily?: string | null,
) => {
Copy link
Member

Choose a reason for hiding this comment

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

Do we want to return a Promise<PAYWALL_RESULT> in this method as well?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

You're right, we should 👍

Comment on lines +13 to +14
_displayCloseButton?: boolean,
_fontFamily?: string | null,
Copy link
Member

Choose a reason for hiding this comment

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

These two are unused. Is it intentional to avoid breaking the API in the future?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yeah, this is basically called from the shared code, and these parameters are sent. I mostly prefer to have them so we know they exist, but are unused currently.

Copy link
Member

Choose a reason for hiding this comment

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

Yep. Makes total sense!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr:other A code change that improves performance

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants