-
Notifications
You must be signed in to change notification settings - Fork 694
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
Added Stripo email editor to tools #2439
Conversation
Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). 📝 Please visit https://cla.developers.google.com/ to sign. Once you've signed (or fixed any issues), please reply here (e.g. What to do if you already signed the CLAIndividual signers
Corporate signers
ℹ️ Googlers: Go here for more info. |
I signed it! |
@fstanis requested you for review according to ampproject/wg-outreach#15 as you're part of the wg-amp4email. |
Approved, but the CLA seems to still be pending. You may have used a different address in your commit, to fix that:
|
Done |
Hello @matthiasrohmer , can we ask you to review the cla/google check again? Thanks |
The CLA is reviewed automatically, neither Matthias nor I can do much here. Go to https://cla.developers.google.com/clas and make sure it's signed with the correct username (@antongrinenko needs to do it, as the author of the PR) and with the same email used in the commits (the one you set with |
Thank you, Filip. Will check with Anton again. |
CLAs look good, thanks! ℹ️ Googlers: Go here for more info. |
@fstanis seems like we did it :) Thanks a lot for the help |
Fixes #2438.