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

[BUG] Incorrect load state signal in Reward Ads #4

Closed
kyoz opened this issue Aug 23, 2021 · 13 comments
Closed

[BUG] Incorrect load state signal in Reward Ads #4

kyoz opened this issue Aug 23, 2021 · 13 comments
Labels
bug Something isn't working
Milestone

Comments

@kyoz
Copy link
Contributor

kyoz commented Aug 23, 2021

Describe the bug

I think this line have incorrect signal, it should be load_fail rather than show_fail.

Just testing it on my iphone and this cause the Toast to repeating show just like when user click on view ads but there is no ads :D

Expected behavior

Should have valid signal

@kyoz kyoz added the bug Something isn't working label Aug 23, 2021
@kyoz kyoz changed the title [BUG] Incorrect load state callback in Reward Ads [BUG] Incorrect load state signal in Reward Ads Aug 23, 2021
@gumaciel
Copy link
Contributor

@kyoz Thank you for reporting it, i will fix it on the next update

@gumaciel gumaciel added this to the 2.0.0 milestone Aug 24, 2021
@kyoz
Copy link
Contributor Author

kyoz commented Aug 25, 2021

Hi @gumaciel , i finally found out. The plugin work perfectly, that's just the ADAPTIVE option. I'v used it in android so i think it will work on iOS too and keep it. But don't know why it doesn't work. I'v changed it to SMART_BANNER and now it working normally on both Simulator and Real Devices. :D

@kyoz
Copy link
Contributor Author

kyoz commented Aug 25, 2021

In XCode i can see it is loaded but don't know why it doesn't show so i don't know it's problem in iOS or on plugin side.

@gumaciel
Copy link
Contributor

Hi @kyoz i created a issue for this, i will try to fix on the next update

gumaciel added a commit that referenced this issue Sep 1, 2021
@gumaciel
Copy link
Contributor

gumaciel commented Sep 1, 2021

@kyoz Hi, i fixed the issue.

You can test on this way:

Or just wait the Version 2.0.0 of iOS Plugin and the 1.0.0 of Editor Plugin be released.

@kyoz
Copy link
Contributor Author

kyoz commented Sep 2, 2021

Hi @gumaciel , I've just build and test it, doesn't know if my build progress is correct but i'v got this error in XCode output @@

**ERROR**: In Object of type 'Node': Attempt to connect nonexistent signal 'rewarded_ad_opened' to method 'CanvasLayer._on_rewarded_ad_opened'.
**ERROR**: In Object of type 'Node': Attempt to connect nonexistent signal 'rewarded_ad_opened' to method 'CanvasLayer._on_rewarded_ad_opened'.
e': Attempt to connect nonexistent signal 'rewarded_ad_closed' to method 'CanvasLayer._on_rewarded_ad_closed'.
**ERROR**: In Object of type 'Node': Attempt to connect nonexistent signal 'rewarded_ad_closed' to method 'CanvasLayer._on_rewarded_ad_closed'.

@kyoz
Copy link
Contributor Author

kyoz commented Sep 2, 2021

Oh i know it, `MobileAds.plugin.connect("signal", self, "func") for now.

After testing i can see the show_fail and load_fail signal is now correct but after user watched, the user_earned_rewarded is not fired 😢.

Also for the banner, the adaptive banner still doesn't show, other banner is all fine. Doesn't know why, but i think i'll use "SMART_BANNER" for ios for now.

@gumaciel
Copy link
Contributor

gumaciel commented Sep 2, 2021

@kyoz please try to use with the Example.tscn: https://github.com/Poing-Studios/godot-admob-editor/blob/f97e553f19dec7cebfa7ff7e4d42f8921eb3c899/addons/admob/develop/Example.tscn

If not works, try to delete the .ipa and export again.

Oh, the ADAPTIVE banner was't been fixed yet, I'll do that on next days.

@kyoz
Copy link
Contributor Author

kyoz commented Sep 2, 2021

Thanks @gumaciel, yes i'v tried which develop folder in godot-admob-editor. Doesn't know why it not fire user_earned_rewarded. Maybe i've builded it wrong. I'll try it again. 😸

@gumaciel
Copy link
Contributor

gumaciel commented Sep 3, 2021

@kyoz you can test with this .zip
admob-ios-dev-2_0_0-1.zip

Also: I just send a new update to Editor Plugin: poingstudios/godot-admob-plugin@007b79a of this Issue: poingstudios/godot-admob-plugin#10

  • Remove the need of "Export -> Resources -> Filters to export non-resources..., you need to put this: res://addons/admob/*.json"
  • Changed the .json to a .gd Script which extends Node and only have a Dictionary Variable to store the config variable, located at: addons/admob/config/Settings.gd
  • Removed addons/admob/config/settings.json due is useless now with the Settings,gd

Note: If change the Variables directly from the Editor and change the screen, then will give this error: Another resource is loaded from path 'res://addons/admob/config/Settings.gd' (possible cyclic resource inclusion). because the Script is not fully reloaded when is saved, this won't change the functionality of the Plugin so it will only need to fix this for next versions.

So now you don't need to put anything here to export
Captura de Tela 2021-09-02 às 23 32 11

@gumaciel
Copy link
Contributor

gumaciel commented Sep 3, 2021

@kyoz Just tested here on the new version of Editor Plugin and iOS Plugin (that i send to you in few minutes ago) and it's working well the user_earned_rewarded:

Captura de Tela 2021-09-02 às 23 44 25

"EARNED coins with amount: 10"

@kyoz
Copy link
Contributor Author

kyoz commented Sep 3, 2021

Hi @gumaciel , yeah just as you said, i'v delete the ipa project and export again and now everything work, both on example develop project and my project. Everything now is work perfectly. The option to auto toggle real ads on production is just awesome.

@gumaciel
Copy link
Contributor

gumaciel commented Sep 4, 2021

Nice, since everything seems to be fixed i will close the Issue, the issue of ADAPTIVE BANNER is here: #6.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants