Just a word to inform that I started working on the port of this module to D8.
I don't say it's going to be quick but if you want to work on it too, please ask :)

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

Comments

duaelfr’s picture

Issue summary: View changes
mbovan’s picture

Hey! Are you still working on the port? Do you have a link to sandbox/github repo?

duaelfr’s picture

Assigned: duaelfr » Unassigned

I'm not working on it anymore. I think I started too soon, so ran into a lot of issues. Plus, I started to write a blog post series to explain how to do, so it took me even more time.
I just pushed my work in the 8.x-2.x branch so if you want to continue, do not hesitate. Again, it's quite old and it's never been over. Don't expect it to "work" on 8.1.x :)

geek-merlin’s picture

My gut feeling is that this is more of a complete rewrite than a straight port if we want to leverage the D8 blessings.

drumm’s picture

D8 core has an email field type. I'm not sure if it can confirm email addresses or not. Multiple email could move to extend a multi-valued email field on user entities.

geek-merlin’s picture

Yah, i did not wrap my head completely around this, but even making the core email field multiple may be feasible.

manuel.adan’s picture

For the upcoming D8 version, the email_confirmer module might be useful. It makes the hard work for any other module that needs email address confirmation as is the case with this one.

pasqualle’s picture

Title: D8 Port » Port Multiple E-mail Addresses to Drupal 8
avpaderno’s picture

Issue tags: -d8port
jonathanshaw’s picture

I created a Drupal 8 module similar to this called 'Alternative User Emails'. It's not as ambitious as this module, but it' might be good enough for 90% of use cases: https://www.drupal.org/project/alternative_user_emails

voleger’s picture

How to contribute to the 8.x version?
Can maintainers define the development branch and enable testing?
This will help to provide some contributions using the separate issue forks.

duaelfr’s picture

Title: Port Multiple E-mail Addresses to Drupal 8 » Port Multiple E-mail Addresses to Drupal 8/9
Version: 7.x-1.x-dev » 2.x-dev

@voleger
Thanks for stepping up!
I renamed the 8.x-2.x branch to 2.x, quickly rebased it on latest 7.x-1.x then pushed and created the dev release and enabled some tests.

voleger’s picture

Great, thanks.

voleger’s picture

Updated the tests by converting them to Functional tests and replaced deprecated function calls in tests.

The next step is to convert hook_menu implementation to routing configuration and update all hook implementation to the compatible implementations (check if they implemented propely).

voleger’s picture

Can be test enabled for the 2.x branch? Also regarding compatibility with semver the development branch better to rename to 2.0.x.
Then switch the target branch in the merge request to the new branch before removal 2.x branch.

davisben made their first commit to this issue’s fork.

davisben’s picture

I'm working on a project where I need this functionality, so I took a stab at finishing up this port. It's not 100%, but it's a good chunk of the way there.

davisben’s picture

Status: Active » Needs review

  • DuaelFr committed 404bbdd on 2.x authored by voleger
    Issue #2489232: Port Multiple E-mail Addresses to Drupal 8/9
    
duaelfr’s picture

Nothing shocked me in the code so I merged the MR.
Thanks a lot for all the work here!

@davisben do you think you covered enough to make a first alpha release? What's remaining to do on this port?

davisben’s picture

It should be in good shape for an alpha. The UI could probably use some cleaning up, but I think all the functionality has been accounted for. I've been using it for a couple months, but I'm just one person with a limited use case.

duaelfr’s picture

Status: Needs review » Fixed

I'll make a release then.
Thanks again a lot for sharing your version with the community!

Status: Fixed » Closed (fixed)

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