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

[HB-5971] Update loading-ads.md to Include More Details On Adaptive Banners #188

Merged
merged 4 commits into from
Oct 12, 2023

Conversation

kushG
Copy link
Contributor

@kushG kushG commented Oct 12, 2023

No description provided.

kushG added 2 commits October 12, 2023 12:33
Signed-off-by: kushG <kushagrag@sturfee.com>
Signed-off-by: kushG <kushagrag@sturfee.com>
@github-actions
Copy link

github-actions bot commented Oct 12, 2023

Test Results

1 tests  ±0   1 ✔️ ±0   0s ⏱️ -3s
1 suites ±0   0 💤 ±0 
1 files   ±0   0 ±0 

Results for commit 00998c3. ± Comparison against base commit c485d78.

♻️ This comment has been updated with latest results.

Copy link

@cinnopuu cinnopuu left a comment

Choose a reason for hiding this comment

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

comments for corrections & additions added to loading-ads.md

kushG added 2 commits October 12, 2023 13:24
Signed-off-by: kushG <kushagrag@sturfee.com>
Signed-off-by: kushG <kushagrag@sturfee.com>
@kushG kushG merged commit ecacb3f into release/4.6.0 Oct 12, 2023
@kushG kushG deleted the kg/adaptive-banner-documentation branch October 12, 2023 20:54
kushG added a commit that referenced this pull request Oct 12, 2023
* Added Missing Meta Files (#172)

* [HB-6516] Create a Separate Size Object for Newer Banner API Instead of Updating the Size from Deprecated API (#173)

* rename ChartboostMediationBannerAdSize to ChartboostMediationBannerSize

* base file didn't get pushed

* missing android update for sizeType enum of ChartboostMediationBannerSize (#174)

* Updating Dependencies and Package Information (#175)

* Remove UnityBannerAd Creation Option from Chartboost Mediation MenuItem (#176)

* Fix For Load With (X, Y) on Unsupported BannerView (#177)

* [HB-6453] Mark `ChartboostMediationBannerAd` API  as Deprecated (#178)

* [HB-6572] Always Keep the BannerAd within the Safe Area (#179)

* [HB- 6580] Fix Resize when `ChartboostMediationBannerView` API is in Use (#180)

* Logger.LogError should use Debug.LogError (#181)

* Trigger DidStart on ChartboostMediationUnsupported (#182)

* [HB-6597]  Adaptive Banner Fixes (#183)

* Fix for crash on ad reset (#184)

Signed-off-by: kushG <kushagrag@sturfee.com>

* Update CHANGELOG.md (#185)

Signed-off-by: kushG <kushagrag@sturfee.com>

* [HB-6599] Adds a `MoveTo` internal API call to sync UnityBannerAd Gameobject position with Native View (#186)

* moveTo added, canvas replaced with parent transform during unitybannerAd create

* demo fix

* [HB-5971]Update the Readme with Instructions and Sample code for Using the Adaptive Banner 3D game object (#187)

* IChartboostMediationBannerView.cs updated

* all the remaining documentation

* Update loading-ads.md

Signed-off-by: kushG <kushagrag@sturfee.com>

---------

Signed-off-by: kushG <kushagrag@sturfee.com>

* minor fix with canvas scale

Signed-off-by: kushG <kushagrag@sturfee.com>

* [HB-5971] Update loading-ads.md to Include More Details On Adaptive Banners (#188)

* Update loading-ads.md

Signed-off-by: kushG <kushagrag@sturfee.com>

* fixed links

Signed-off-by: kushG <kushagrag@sturfee.com>

* Update loading-ads.md

Signed-off-by: kushG <kushagrag@sturfee.com>

* Update loading-ads.md

Signed-off-by: kushG <kushagrag@sturfee.com>

---------

Signed-off-by: kushG <kushagrag@sturfee.com>

---------

Signed-off-by: kushG <kushagrag@sturfee.com>
Co-authored-by: Santiago Castaneda Munoz <santiagocm98@hotmail.com>
kushG added a commit that referenced this pull request Oct 12, 2023
* ios updated to use adaptive banners

* WIP banner work for 5.0

* Fixed banner load working on ios

* android updates

* android updates

* more android update

* banner keywords iOS WIP

* ios keywords fix

* renaming variables

* android updates

* working android code for fixed banners

* keeping old API unchanged

* code for onAdRefreshed added

* AdWrapper fix for deprecated APIs

* android bridge updates

* removing unwanted files from git

* destroy added

* size name made get only

* working gameobject code for android for fixed banners

* creator added

* working android and ios e2e

* onAdViewAdded added on android

* minor changes

* UnityBannerAdSize added

* Editor code added

* keys commit removed

* added missing files

* [HB-6464] Updating EDM to version 1.2.177 (#169)

* added missing files

* Banner drag fix

* Update com.chartboost.mediation/Runtime/Platforms/IOS/ChartboostMediationIOS.Callbacks.cs

Co-authored-by: Kendall <60120337+kendallrogers@users.noreply.github.com>
Signed-off-by: kushG <kushagrag@sturfee.com>

* default case added in size names for ios

* using screen width for adaptive banners width

* WrapperStore added in iOS

* Update build.gradle

Signed-off-by: kushG <kushagrag@sturfee.com>

* fix for horizontal and vertical alignment on android

* Apply suggestions from code review

Co-authored-by: Ray Graham <rayg@bitwisedesign.com>
Signed-off-by: kushG <kushagrag@sturfee.com>

* formatting updates, minor changes based on review comments

* more formatting changes

* testing line end commit

* Extra blank lines at the end of file on android code in Rider

* Update com.chartboost.mediation/Runtime/AdFormats/Banner/Unity/ChartboostMediationUnityBannerAd.Creator.cs

Co-authored-by: Santiago Castaneda Munoz <santiagocm98@hotmail.com>
Signed-off-by: kushG <kushagrag@sturfee.com>

* Apply suggestions from code review

Co-authored-by: Santiago Castaneda Munoz <santiagocm98@hotmail.com>
Signed-off-by: kushG <kushagrag@sturfee.com>

* few more updated based on review comments

* not destroying bannerAd in demo's OnDestroy

* Fixes missing/extra EOL

* removed double code of setting banner ad to visible

* minor fix in gameobject naming

* finding canvas made better

* WillAppear changed to DidLoad

* [HB-6495] Refactor ChartboostMediationBannerAdSize to be used as Struct and use Enum for Name Field (#170)

* size refactor

* ChartboostMediationBannerAdSize changed from class to struct

* ChartboostMediationBannerName renamed to ChartboostMediationBannerSizeType

* minor rename

* [HB-6514 ] Include API to Allow Resizing of Gameobject Banner Based on Adaptive Banner Size (#171)

* resize API added

* removed #UNITY_ANDROID

* WrapperStore added in iOS

* resize working on iOS

* banner request container removed in iOS

* android WIP

* working logic for resize and drag on android

* fix for horizontal and vertical alignment on android

* size refactor

* ChartboostMediationBannerAdSize changed from class to struct

* ChartboostMediationBannerName renamed to ChartboostMediationBannerSizeType

* merge fixes

* minor rename

* working resize and drag on android

* working ios code for resize and drag

* DiscardOversizedAds added

* removed debug code

* fixes for null/default values and removed some debug code

* default state of banner drag set ot false

* default state of resize set to "FitBoth"

* missing EOL

* autoLoadOnInit removed

* default set to AllNetworkBanner in demo

* renaming some enum options

* disable test mode

* Added Missing Meta Files (#172)

* [HB-6516] Create a Separate Size Object for Newer Banner API Instead of Updating the Size from Deprecated API (#173)

* rename ChartboostMediationBannerAdSize to ChartboostMediationBannerSize

* base file didn't get pushed

* missing android update for sizeType enum of ChartboostMediationBannerSize (#174)

* Updating Dependencies and Package Information (#175)

* Remove UnityBannerAd Creation Option from Chartboost Mediation MenuItem (#176)

* Fix For Load With (X, Y) on Unsupported BannerView (#177)

* [HB-6453] Mark `ChartboostMediationBannerAd` API  as Deprecated (#178)

* [HB-6572] Always Keep the BannerAd within the Safe Area (#179)

* [HB- 6580] Fix Resize when `ChartboostMediationBannerView` API is in Use (#180)

* Logger.LogError should use Debug.LogError (#181)

* Trigger DidStart on ChartboostMediationUnsupported (#182)

* [HB-6597]  Adaptive Banner Fixes (#183)

* Fix for crash on ad reset (#184)

Signed-off-by: kushG <kushagrag@sturfee.com>

* Update CHANGELOG.md (#185)

Signed-off-by: kushG <kushagrag@sturfee.com>

* [HB-6599] Adds a `MoveTo` internal API call to sync UnityBannerAd Gameobject position with Native View (#186)

* moveTo added, canvas replaced with parent transform during unitybannerAd create

* demo fix

* [HB-5971]Update the Readme with Instructions and Sample code for Using the Adaptive Banner 3D game object (#187)

* IChartboostMediationBannerView.cs updated

* all the remaining documentation

* Update loading-ads.md

Signed-off-by: kushG <kushagrag@sturfee.com>

---------

Signed-off-by: kushG <kushagrag@sturfee.com>

* minor fix with canvas scale

Signed-off-by: kushG <kushagrag@sturfee.com>

* [HB-5971] Update loading-ads.md to Include More Details On Adaptive Banners (#188)

* Update loading-ads.md

Signed-off-by: kushG <kushagrag@sturfee.com>

* fixed links

Signed-off-by: kushG <kushagrag@sturfee.com>

* Update loading-ads.md

Signed-off-by: kushG <kushagrag@sturfee.com>

* Update loading-ads.md

Signed-off-by: kushG <kushagrag@sturfee.com>

---------

Signed-off-by: kushG <kushagrag@sturfee.com>

---------

Signed-off-by: kushG <kushagrag@sturfee.com>
Co-authored-by: Santiago Castaneda Munoz <santiagocm98@hotmail.com>
Co-authored-by: Kendall <60120337+kendallrogers@users.noreply.github.com>
Co-authored-by: Ray Graham <rayg@bitwisedesign.com>
@SCastanedaMunoz SCastanedaMunoz added the documentation Improvements or additions to documentation label Oct 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants