Skip to content

Clarify rewarded video availability check timing#1540

Closed
jk7287 wants to merge 4 commits into
mainfrom
docs/defer-ad-availability-check
Closed

Clarify rewarded video availability check timing#1540
jk7287 wants to merge 4 commits into
mainfrom
docs/defer-ad-availability-check

Conversation

@jk7287

@jk7287 jk7287 commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

Changes

This surfaces the existing GetAdAvailabilityNowAsync timing guidance next to the client implementation steps. Developers should call the method as close as possible to showing the ad, such as when the user opens the shop menu containing the video ad button.

The best practices now also avoid calling GetAdAvailabilityNowAsync after an ad finishes unless another ad is being offered.

Checks

By submitting your pull request for review, you agree to the following:

  • This contribution was created in whole or in part by me, and I have the right to submit it under the terms of this repository's open source licenses.
  • I understand and agree that this contribution and a record of it are public, maintained indefinitely, and may be redistributed under the terms of this repository's open source licenses.
  • To the best of my knowledge, all proposed changes are accurate.

@github-actions github-actions Bot added the engine guides Changes the Engine guides label Jul 15, 2026

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Hi @jk7287, thanks so much for helping improve the Roblox creator documentation! Our technical writing team will review your pull request soon. In the meantime, please ensure you've read through the README.md, contribution guidelines, and style recommendations.

@jk7287 jk7287 closed this Jul 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

engine guides Changes the Engine guides

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant