Closed alfaro28 closed 1 month ago
@Lovedanihonjin we updated the image, please review again
Hi @Lovedanihonjin could you please review again?
Hi @Lovedanihonjin could you please review again?
Hi @Lovedanihonjin , I would like to know an estimation of when is this PR going to be merged?
Hi @Lovedanihonjin is there anything we still need to do to have this PR merged?
Hi @Lovedanihonjin I would just like some visibility on why this PR is not being merged despite being approve, and some visibility on an approximate date of release
Hi @alfaro28 , are you able to check the conflicts for this PR? I am unable to merge it due to an error that needs to be resolved via the command line
Hi @dannye0231 , I fixed the conflicts
Hi @dannye0231 , do you have an estimated date for the next release?
Hi @alfaro28 , we are planning to do another run this Wednesday. You should see it merged into Prod by the end of this week.
Why are you adding this icons?
I'm adding/updating this icon(s) because Banco Azteca icon is missing and we want to use it for our payment app
Help us identify yourself
Link to the brand guidelines:
Checklist to add new icons
db/payment_icons.yml
If this pull request is not adding new icons, you can remove this checklist.
Attach a screenshot of the icon along side the example Visa icon
If the icons are intended for use by Shopify, please provide the following info:
Who are you working with at Shopify? (avoid adding personal details, provide github handle(preferred) or first name and last name)
What's the expected date of this change to deploy on Shopify?