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

Ads repo fixbugs #239

Merged
merged 6 commits into from
Dec 26, 2021
Merged

Ads repo fixbugs #239

merged 6 commits into from
Dec 26, 2021

Conversation

Ali72
Copy link
Contributor

@Ali72 Ali72 commented Dec 18, 2021

changes:

  • fix android bugs

    1. failed event not called
    2. concurrency problem fix
    3. remove request after ads load failed
  • fix ios bugs

    1. add retry system for ad repository
    2. fix retry handler after failed
    3. remove image RNGADImageView
      case: the previous developer add RNGADImageView for this bug but after this update, the bug was fixed and no need to create RNGADImageView object anymore. on the other hand, if use RNGADImageView, it makes some bug in the project when the image load fast before the set ImageView size.

Tip: You can make two version of your repository one for before version 0.63.2 and one after version 0.63.2.
Note: Except remove image RNGADImageView, This update is required and necessary

* handle retry from repository (instead of ad view)
* ad listener for android changed (for: click, impression, open, closed)
* remove GADImageView from IOS (image will load with image tag in react-native)
@ammarahm-ed
Copy link
Owner

ammarahm-ed commented Dec 26, 2021

Is it possible that we keep using RNGADImageView for older RN Versions but use Image in newer RN versions? It will be difficult to maintain two separate versions of the same library. Or we can let this be and if anyone has an issue, they can upgrade their RN version

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