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: (YouTube) Enabling ads doesn't enable ads (solution depends on changing another setting) #3103

Closed
3 tasks done
flanter21 opened this issue May 1, 2024 · 6 comments · Fixed by #3130 or #3053
Closed
3 tasks done
Labels
Bug report Something isn't working

Comments

@flanter21
Copy link

Bug description

I was having issues enabling video ads in revanced. Note that I get a lot of ads on my phone and my laptop so I don't think its because of where I live or smthg.

What I tried

  • Enabling all ads in revanced settings
  • Reinstalling using revanced manager disabling the ads patches

Error logs

No response

Solution

Eventually I found out that if you turn off

  • Parameters → ReVanced → General Layout → Navigation buttons → Switch create with notifications
    it works again.

I suggest that this setting is automatically turned off when someone disables video ads, or that the person is given a popup telling them to do so, so that they aren't confused when ads don't appear anyways.

Additional context

No response

Acknowledgements

  • This issue is not a duplicate of an existing bug report.
  • I have chosen an appropriate title.
  • All requested information has been provided properly.
@flanter21 flanter21 added the Bug report Something isn't working label May 1, 2024
@kazimmt
Copy link
Contributor

kazimmt commented May 1, 2024

It's not a bug
Switch create with notifications spoofs YT to android auto version. & Android auto version doesn't have those ads.

@LisoUseInAIKyrios
Copy link
Contributor

The hide ads setting can show a dialog message if the user tries to turn it off.

@flanter21
Copy link
Author

It's not a bug Switch create with notifications spoofs YT to android auto version. & Android auto version doesn't have those ads.

It may not be a bug, but it makes the enable ads setting useless if you don't turn it off. My issue is that it isn't clear that you must turn it off. I certainly had no idea that this setting was blocking ads before a lot of trial and error. The setting description itself also doesn't mention that it blocks ads, so even if someone finds this setting, which isn't in the ads menu, they would have no idea they have to toggle it or that it has any relation to ads at all.

@oSumAtrIX
Copy link
Member

@LisoUseInAIKyrios both patches are separate so the ads the patch can't display a dialog about the other patch since it doesn't know anything about it

@LisoUseInAIKyrios
Copy link
Contributor

Yeah that is true.

I suppose the best (simple) solution is to add a text description to the switch notifications setting summary.

@flanter21
Copy link
Author

@LisoUseInAIKyrios both patches are separate so the ads the patch can't display a dialog about the other patch since it doesn't know anything about it

Maybe you could change the text under “switch create with notifications” to say that it also disables video ads? That way at least it’s visible somewhere and makes the description more accurate.

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