Manuel Kaufmann
Manuel Kaufmann
> We aren't removing that capability, but the cross site cookies needed to make that work do not work on Safari or Firefox. We'll leave the functionality in for Chrome...
Oh, I'm sorry. I miss read the issue. **You are removing ads from your project in particular**. OK, OK. I'm re-opening it since I'm not sure about the decision we...
> We discussed this and it makes sense to apply this to subprojects You made a typo here. We want to apply this (removing ads) to _translations_ and **not** subprojects...
Thanks for opening this issue. This would be a great validation 👍
We can move the removal of that particular test to another PR if we want, so we can merge that removal quicker.
I'm adding this PR to the next sprint since we are all already in addons. Besides updating this PR, we will want to [remove the this header from the CF...
Is this required? I understand we have some customer already using `build-large` queue, but I don't understand how they are using the `build-large` queue without this router 🤔
> I actually like this implementation a lot, since it makes API v3 the same across our sites, which feels like a huge simplification 💯 Agreed! 👍🏼
Blocked by https://github.com/readthedocs/readthedocs-corporate/pull/1873
> Should the comment also respect the ignore patterns from the addon config? It's also kind of confusing that we have these settings in two places... in addons and in...