Skip to content
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

[google_adsense_ad_placement_api_web] Creates new package to simplify interaction with AdSense Ad Placement API for Flutter Apps #7747

Draft
wants to merge 22 commits into
base: main
Choose a base branch
from

Conversation

astivi
Copy link

@astivi astivi commented Oct 1, 2024

Official library release to simplify monetization of Flutter Apps by providing an easy to use interface to the AdSense Ad Placement API in the Web platform.
Please note that this package only supports interacting with the Ad Placement API after it's already loaded. Before using it remember to include the appropriate adsense js files.

List which issues are fixed by this PR. You must list at least one issue.

Pre-launch Checklist

If you need help, consider asking for advice on the #hackers-new channel on Discord.

@astivi astivi changed the title [google_adsense_ad_placement_api] Creates new package to simplify interaction with AdSense Ad Placement API for Flutter Apps [google_adsense_ad_placement_api_web] Creates new package to simplify interaction with AdSense Ad Placement API for Flutter Apps Oct 1, 2024
@ditman
Copy link
Member

ditman commented Oct 28, 2024

Adding myself to this so I can take a look! @astivi, is this something you want to merge or are still working on it?

@astivi
Copy link
Author

astivi commented Oct 28, 2024

Adding myself to this so I can take a look! @astivi, is this something you want to merge or are still working on it?

Hey there @ditman,
I'll probably join these changes on #6871 after that is merged in! Don't worry about reviewing this for now. Thanks!

@ditman
Copy link
Member

ditman commented Oct 29, 2024

I'll probably join these changes on #6871 after that is merged in! Don't worry about reviewing this for now. Thanks!

I was going to suggest that, or at least to share some code (for example the installing of the JS library). Thanks!

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