Issam Mani
Issam Mani
@nbhasin2 @jnrahme this change can only be merged after this PR https://github.com/mozilla/application-services/pull/6118 is merged, but a-s recommends to get approval before merging the upstream change.
@nbhasin2 @razvanlitianu this is what I talked about last time with preloading. This is a common pattern on the web but this is my first time implementing it in swift....
> @issammani the tests are passing but BR is acting up, let me know if you would like me to hit merge on this Yes please. Thanks !
> @issammani Can I get your 👀 on this? It looks like we reference these OAuth scopes in 3 places. One is called during a new sign-in, one is called...
Thanks @Almaju. I can take a look at the AS bits next week. Tagging @lmarceau since she did touch the ads code a while back and has context.
@Almaju merged https://github.com/mozilla-mobile/firefox-ios/pull/31310. You shouldn't need the changes in the `Generated/` files anymore