Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Let's revive Simplenews for Backdrop! #1

Closed
alanmels opened this issue Nov 7, 2019 · 2 comments
Closed

Let's revive Simplenews for Backdrop! #1

alanmels opened this issue Nov 7, 2019 · 2 comments

Comments

@alanmels
Copy link
Member

alanmels commented Nov 7, 2019

Hi Andy,

I am Alan of AltaGrade, where our team had started to port Simplenews, unfortunately, without first checking if any previous attempt was ever taken. I believe that's because I've read on https://gitter.im/backdrop/backdrop-issues that Simplenews was needed by the community and assumed there was no such a project on Backdrop yet.

So only after we finished porting the module and publishing it for now on https://github.com/altagrade/simplenews, I was pleasantly surprised that https://github.com/backdrop-contrib/simplenews was already taken. Pleasantly because I indeed thought it's really great that someone else was also interested in getting the module work on Backdrop.

The readme file on https://github.com/backdrop-contrib/simplenews says the project is seeking maintainers. Why don't we join efforts and start maintaining this project together? Let me know if you are interested in reviving Simplenews for Backdrop.

@quicksketch
Copy link
Member

quicksketch commented Nov 8, 2019

Thanks @alanmels, per backdrop-ops/contrib#373 I've approved your request to maintain this module.

Because it looks like your port takes a different approach in some ways (the names of config variables for example), the port is not compatible with the current implementation. Had this module had a full (non-alpha) release, I would suggest making a version 2.0 instead. But as-is, your port taking over the 1.x namespace may be okay. I would encourage you to write an upgrade path if possible though for existing users.

Thanks @biolithic for initially creating this module. I hope you'll approve of the work @alanmels and other contributors provide for this module in the future.

@alanmels
Copy link
Member Author

alanmels commented Nov 8, 2019

@quicksketch, Thank you for your review and approval of my request.

Nice idea for upgrade path. Unfortunately, at the moment the hands of our team are tied up to current tasks queue on AltaGrade. However, if somebody in posterity will hit issues with upgrade we will help them or quickly write up the upgrade path.

@biolithic, you are more than welcome to join in as co-maintainer of the module any time.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants