search

maximnara / cordova-plugin-yandex-ads

Yandex Ads for Cordova apps
MIT License
16 stars 7 forks source link

remove mytarget mediation #5

Closed Junker closed 1 year ago

Junker commented 1 year ago

Moved MyTarget mediation to the separate plugin: https://github.com/Junker/cordova-plugin-yandex-ads-mediation-mytarget

maximnara commented 1 year ago

Не могу это принять в текущем виде: части пользователей нужна медиация, без нее не работают. Мне кажется базово можно закомментировать инжект pod, activity оставить.

Вообще если вам это не нужно — можно просто локально закомментить.

Junker commented 1 year ago

Я думаю, что данный плагин должен содержать только код, который работает с РСЯ, а медиации должны в отдельных плагинах. Медиаций много: https://yandex.ru/dev/mobile-ads/doc/android/mob-mediation/list-network.html Почему в этом плагине только MyTarget медиация? Нужно вносить все, либо никакие. Не всем нужна медиация MyTarget, лучше ее сделать опциональной, через дополнительный плагин, который будет работать в связке: https://github.com/Junker/cordova-plugin-yandex-ads-mediation-mytarget

Вот пример как это работает с AdMob'ом: есть основной плагин admob-plus, который работает с AdMob сетью: https://github.com/admob-plus/admob-plus и отдельные дополнительные плагины для медиаций, которые работают в связке с admob-plus: https://github.com/yjseo29/cordova-plugin-admob-mediation-facebook https://github.com/Junker/cordova-plugin-admob-mediation-applovin https://github.com/Junker/cordova-plugin-admob-mediation-adcolony и т.д.

maximnara commented 1 year ago

Не могу принять этот пул реквест, пусть будет медиация.