Hi there,

started yesterday porting the 7.x-2.x version to drupal 8
here's the git repo https://github.com/iwanfux/simplenews/tree/8.x-1.x

best regards
ifux

Roadmap:

  1. Convert settings forms
  2. Newsletter config entity
  3. Subscriber content entity (Including edit form but probably no views integration yet)
  4. Subscriptions as an extended entity reference between subscribers and newsletters
  5. Roughly at this point, start to port the pretty extensive test coverage that we have together with porting the functionality
  6. Subscription blocks and pages
  7. Newsletter issues (Extended entity reference to newsletters for nodes)
  8. Spool management
  9. Sending of newsletters
  10. Port mailing tests
  11. Views integration (Including replacing the default subscriber and newsletter issue listings with views)
  12. Migrate path
  13. Displaying recent issues in subscription blocks
  14. Everything that I forgot and new features :)

Comments

miro_dietiker’s picture

Awesome to have you part of the team and getting started! We are always available for feedback.

Could you please update this post with a bit of status and strategy detail (next steps / short term goals) on a periodical basis?
The finally ultimate thing would be a roadmap with a timeline. ;-)

berdir’s picture

Issue summary: View changes

Yeah, we're going to make a contrib maintainer out of you ;)

Started a roadmap in the issue summary.

ifux’s picture

Issue summary: View changes
ifux’s picture

Issue summary: View changes
ifux’s picture

Issue summary: View changes
ifux’s picture

Issue summary: View changes

Finished a big part of the test coverage (settings, subscriptions)
Ported subscription blocks and pages.

miro_dietiker’s picture

Thank you for the great progress recently and we are happy to have you join our sprints!

As discussed, we recently stumbled upon personalisation again and actually with every single project this topic came up. (And some strange hacks / custom modules where required since simplenews is not really made for it out of the box...) I do like to address this issue with our Drupal 8 release as early as possible.

Possibly missing above is a migrate path and in reference to the issue about the status of 7.x-2.x the question if we support it at all for migration or if we recommend to migrate from 7.x-1.x.

miro_dietiker’s picture

Issue summary: View changes

Added migrate path to summary ;-)

berdir’s picture

With migrations, it would certainly be easier to support both, but there's a lot of stuff that yet needs to be figured out. The default migration would automatically convert the taxonomy terms that we no longer need for example, but I guess you could delete them manually.

There are few reasons to do something special with the target, what we need to provide are sources for all variants that we want to support, process plugins and migration entities to tie it all together.

miro_dietiker’s picture

We have subscriber management working and we're pretty near to Beta.
It's time to have a 8.x branch in Simplenews and a dev release to make this progress visible for the d.o module search and overview pages.
As such i would not wait for the feature "sending newsletters" to complete to merge it in.
As a result, switching to a strict issue based patch + review workflow would be best for future improvements.
What do you think, when is the right time to switch?

berdir’s picture

I think it's too early for that. IMHO doesn't make sense to do that before most of the existing tests are working again, I can review on github too. patch based processes are too slow for the current phase and testbot is broken anyway it seems.

miro_dietiker’s picture

Understand. Please still reconsider before DrupalCon (Contrib Status) though. :-)

ifux’s picture

Issue summary: View changes

Finished porting the test coverage for settings and subscription
Created issues (extended entity reference) and started working on the mail spool stuff

miro_dietiker’s picture

Great progress.

As discussed today:
In Simplenews 7.x we have naming issues due to historic reasons. We thought this was corrected in 7.x-2.x but most importantly for 8.x, we should decide for a clean naming and strictly stick with it. Newsletter, Issue, ...

Also, i would love to have simplytest.me integration ASAP for demo purposes. We push this with all our modules to lower the barriers and people can learn about the module by example. :-)

ifux’s picture

there is another issue we have to work on.
at the moment there is no real subscriber AND subscription overview
its a kind of a mix and we do not have for example an overview of unsubscribed subscribers

miro_dietiker’s picture

As discussed, we introduce the concept of a subscriber.
In order to manage subscribers, we have put subscriber management under people.
Similar to Configuration of "Account settings", there should be a "Subscriber settings". The subscriber field management should be there.
The Subscriber settings will then host settings that maintain the relationship between subscribers and users.
This leads to a pretty clean UI / consistent D8 core UX patterns.

ifux’s picture

Issue summary: View changes

Ported almost all tests (expect block testing)
Mails can now be send

spool management done

miro_dietiker’s picture

Session about Simplenews port status prepared for DrupalCon, together with ifux.

I also requested collaboration with the more recent added newsletter module...
#2346641: Drupal 8 port
Would be great if we find a collaborative consensus for Drupal 8.

BTW: Arla implemented field sync between Subscriber and User.
We are looking forward to merge in full personalisation this week!

Also added an issue about simplytest.me demo data i do like to see:
#2346837: Support simplytest.me

ifux’s picture

Issue summary: View changes

found something which did not directly port
just mentioned it, that we do not forget it :)

miro_dietiker’s picture

I vote for merging this project in to d.o and start working issue queue based.
What do you think?

berdir’s picture

Pushed the current code to drupal.org, after some further cleanup. Commented out a handful of failing tests as drupal.org testing is only useful if the branch is green. Opened #2401019: Enable commented out tests in 8.x-1.x as a reminder to uncomment them again.

I would suggest to create follow-up issues for the remaining issues and then close this issue.

https://www.drupal.org/node/2401069

https://qa.drupal.org/pifr/test/937688 came back green!

berdir’s picture

Version: 7.x-2.x-dev » 8.x-1.x-dev
miro_dietiker’s picture

Awesome!

Will test it soon and start creating issues. :-)

miro_dietiker’s picture

Unassigning ifux from this task.
Opening this task and all followups to other contributor from team and community.

Anushka-mp’s picture

Issues created as discussed. uploading a patch to remove todo.txt

  • Berdir committed 710094e on 8.x-1.x authored by Anushka-mp
    Issue #2254957 by Anushka-mp: Remove todo.txt, replaced with issues.
    
berdir’s picture

Status: Active » Fixed

Great. The only task here that did not have an issue yet was migration support, we created that as well.

Marking this as fixed, there is of course still a lot to do, but we will track further progress in separate issues.

Status: Fixed » Closed (fixed)

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