- Issue created by @BramDriesen
- First commit to issue fork.
- @viren18febs opened merge request.
- Status changed to Needs review
about 1 year ago 11:56am 31 October 2023 - 🇮🇳India viren18febS
HI @BramDriesen
I have uploaded the logo image, please review.
- Status changed to Needs work
about 1 year ago 12:35pm 31 October 2023 - 🇧🇪Belgium BramDriesen Belgium 🇧🇪
While you technically did what was asked, it doesn't look great 😅
I guess this is the right time to upgrade to the latest version of the logo as well!
- 🇮🇹Italy apaderno Brescia, 🇮🇹
Yes, the logo in comment #6 is better.
- 🇮🇪Ireland lostcarpark
If I understand correctly the logo currently in use is the Microsoft Graph API logo. However, this module uses Graph API as the Drupal mailer. Using the current logo alone could imply it is a module for interfacing with Graph API more generally. How about something that combines a mail element with the Graph API logo to give a clearer indication of its function:
This logo uses elements of a logo I previously created for Symfony Mailer module, but adapted for Graph API.
- Status changed to Needs review
about 1 year ago 10:52pm 31 October 2023 - 🇮🇹Italy apaderno Brescia, 🇮🇹
@lostcarpark That is definitively amazing!
I agree: It must be clear this is a module to send emails using Microsoft Graph API, not a generic module that makes requests through the Microsoft Graph API. -
apaderno →
committed dff87a95 on 2.0.x
Issue #3398011 by viren18febS, apaderno, BramDriesen, lostcarpark: Add...
-
apaderno →
committed dff87a95 on 2.0.x
-
apaderno →
committed 78c922f8 on 1.0.x
Issue #3398011 by viren18febS, apaderno, BramDriesen, lostcarpark: Add...
-
apaderno →
committed 78c922f8 on 1.0.x
- Status changed to Fixed
about 1 year ago 12:39pm 1 November 2023 Automatically closed - issue fixed for 2 weeks with no activity.