Problem/Motivation

Related to #3329936: Add a Project Logo, I emailed the Symfony to enquire about using the Symfony logo in a project logo. I received the following reply:

Using the logo as you did is fine (but you should also add the Drupal logo to avoid confusion -- see the next point), but the project name is not. Right now, the name is confusing as people might think it's the "main" Symfony Mailer component, where as it's just the integration with Drupal.

I can see their point, as I didn't initially realise there was a Symfony Mailer project as part of Symfony.

Steps to reproduce

Proposed resolution

Consider modifying the project name. Some possibilities:

  • Symfony Mailer Integration
  • Symfony Mailer for Drupal

I would assume the "symfony_mailer" machine name shouldn't be a problem.

Remaining tasks

User interface changes

API changes

Data model changes

CommentFileSizeAuthor
#17 symfony_mailer.project-name.3335010-interdiff-15-17.txt646 bytesadamps
#17 symfony_mailer.project-name.3335010-17.patch2.49 KBadamps
#15 symfony_mailer.project-name.3335010-15.patch1.77 KBadamps

Comments

lostcarpark created an issue. See original summary.

adamps’s picture

adamps commented
Title: Consider project name in relation to Symfony trademark guidelines » Change project name in relation to Symfony trademark guidelines
Category: Feature request » Task
Status: Needs work » Active

Thanks

adamps’s picture

adamps commented

Consider modifying the project name. Some possibilities:

Symfony Mailer Integration
Symfony Mailer for Drupal

Both are sensible ideas, but also a bit long. We end up writing something like "The Symfony Mailer for Drupal module uses the Symfony Mailer library"...

So I wonder about something short and sweet such as "S-Mailer"??

adamps’s picture

adamps commented
Priority: Normal » Major
lostcarpark’s picture

lostcarpark commented

Sorry for bringing this on you. Probably better to catch it before a stable release.

avpaderno’s picture

avpaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
commented
Issue summary: View changes
jaypan’s picture

jaypan
Location Victoria, BC
commented

What about "Drupal Symfony Mailer"?

lostcarpark’s picture

lostcarpark commented

I do feel keeping Symfony Mailer in the name would help make it clear that it's integrating the Symfony Mailer project, and also keep the recognition that the project has already amassed. I think it just needs to be distinguishable from the Symfony Mailer project that's part of Symfony.

adamps’s picture

adamps commented

Probably better to catch it before a stable release.

Yes indeed, thank you for revealing the problem.

What about "Drupal Symfony Mailer"?

That would work.

adamps’s picture

adamps commented

I changed the project page.

Probably we should also have a patch to change the 2 info.yml files. I can't think of anything else that would need changing.

adamps’s picture

adamps commented

I changed the documentation guide.

jaypan’s picture

jaypan
Location Victoria, BC
commented

The Swiftmailer module also needs to be updated, as it tells people that it Swiftmailer not longer supported on the requirements page of installations, and to switch to Symfony Mailer. That should be updated.

lostcarpark’s picture

lostcarpark commented

I would suggest adding a line to the module description, possibly below the opening paragraph, containing something like the following:

This module was formally named Symfony Mailer, but has been renamed Drupal Symfony Mailer to avoid confusion with the Symfony project of the same name that this module integrates with.

adamps’s picture

adamps commented

#12 and #13 done

adamps’s picture

adamps commented
Status: Active » Needs review
StatusFileSize
new symfony_mailer.project-name.3335010-15.patch1.77 KB

Status: Needs review » Needs work

The last submitted patch, 15: symfony_mailer.project-name.3335010-15.patch, failed testing. View results

adamps’s picture

adamps commented
Status: Needs work » Needs review
StatusFileSize
new symfony_mailer.project-name.3335010-17.patch2.49 KB
new symfony_mailer.project-name.3335010-interdiff-15-17.txt646 bytes

  • AdamPS committed d0f18f36 on 1.x 
    Issue #3335010 by AdamPS: Change project name in relation to Symfony...
adamps’s picture

adamps commented
Status: Needs review » Fixed
jungle’s picture

jungle
He/Him
Location Chongqing, China
commented

Should the change be made to https://www.drupal.org/project/symfony_mailer_log accordingly? Sorry for commenting on a fixed issue.

adamps’s picture

adamps commented

Should the change be made to https://www.drupal.org/project/symfony_mailer_log accordingly?

Yes I would say so. Please raise an issue for that module - I'm not a maintainer.

jungle’s picture

jungle
He/Him
Location Chongqing, China
commented

Re #21 Done, and patch uploaded there, see #3337255: Change project name to Drupal Symfony Mailer Log, thanks!

jungle’s picture

jungle
He/Him
Location Chongqing, China
commented

Hi, @AdamPS, #3337255: Change project name to Drupal Symfony Mailer Log is in. Please change its project name in the Related modules section of this project's project page.

adamps’s picture

adamps commented

Please change its project name in the Related modules section of this project's project page.

Done thanks

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.