Please port this module to D6
| Comment | File | Size | Author |
|---|---|---|---|
| #34 | 6.x-1.0-alpha2.patch | 838 bytes | ccprog |
| #33 | 6.x-1.0-alpha2.patch | 1.48 KB | ccprog |
| #23 | simplenews_scheduler-6.x-1.0-alpha2.zip | 4.99 KB | sgabe |
| #22 | simplenews_scheduler-6.x-1.0-alpha1.zip | 4.54 KB | sgabe |
| #15 | simplenews_scheduler_d6.zip | 13.5 KB | gagarine |
Comments
Comment #1
Babalu commentedsubscribing
Comment #2
eranglr commentedsubscribes
Comment #3
trophaeum commentedsubscribing
Comment #4
capellicYes, please! THANKS!
Comment #5
andypost+1
Comment #6
skstarkiller commentedadd me to the requesters
Comment #7
kenorb commented+1
Comment #8
cardentey commented+2
Comment #9
juan_manuel commentedsubscribes
Comment #10
encho commentedAny news?
Comment #11
dkruglyak commentedAnyone working on this?
Comment #12
chawl commented+1000
Comment #13
mnp commentediam also waiting for simplenewssheduler 6 version
Comment #14
asc.storm commented+1
Comment #15
gagarine commentedA first version created with coder and deadwood module.
They are a lot of code formating problem but my eclipse crash so I have no IDE to format all the code right now.
TODO:
- It's complety untesting...
- use shema function to create DB table in installation
- code formating
Please test it :)
Comment #16
gagarine commentedComment #17
mnp commentedthanks for the d6 port
but there is a bug
i selected a date,time in node edit page and saved it.
agian i edited the page the date is not set properly it is showing jan,1,1970,5:30 the default date
any configuration settings............
Comment #18
techninja commentedsubscribe
Comment #19
Anonymous (not verified) commentedSubscribing.
Sounds like a great module.
Any ETA on porting?
TIA
Comment #20
tobiberlinI also want to subscribe. Unfortunately I have no programming skills to contribute anything... :-/
Comment #21
gagarine commented@mnp thanks to test the module
I don't need this module for me directly, it's just for a friend. This is not a first priority for me so feel free to assign the task to you. I certainly work in this task in some weeks when i can test it with my friend on a real project (better to find bug).
Thanks to understand ;)
Comment #22
sgabe commentedI think I made some progress on this, please try it and give feedback.
Comment #23
sgabe commentedI made some changes and bug fixes since yesterday based on Simplenews 6.x branch.
Now it seems to be a working alpha release, but it needs review.
Please help test it, and report back!
Comment #24
ferrangil commentedSubscribing. I'll try to test it out asap!
Comment #25
asc.storm commentedGot this error...
warning: gmmktime() expects parameter 6 to be long, string given in /Volumes/arsenal/localhost/ntrst/sites/all/modules/simplenews_scheduler/simplenews_scheduler.module on line 141.
Comment #26
probocop commentedSubscribing.
Comment #27
Max_Headroom commentedFeed back
I've been testing it for over a week on Drupal 6.13 with a few members.
Works fine.
Next step is to add more members.
I think this should be uploaded so that more people can test.
Comment #28
javamandk commentedSubscribing
Comment #29
Aerodynamo commentedSubscribing
Let's get this thing working ASAP! I am willing to test it and I am uploading it now!
Comment #30
tobiberlinHi,
I got a error message every time the newsletter is sent by schedule:
I don't see any duplicate entry, the url alias is set correctly to newsletter/taegliches-update-test-edition-X.
Greez,
Tobias
Comment #31
weseze commentedI've installed the module from #23 and it does absolutely nothing for me... no settings, no extra options, no extra fieldset for scheduling, nothing...
Drupal 6.13
Simplenews RC6
Date 2.3
(and a whole bunch of other modules)
Is there anything I need to do to get this working? Anything I might have missed?
Comment #32
chawl commentedConfirming that port works for D 6.13. Settings come when creating/editing a simplenews node by the way, not to anywhere else.
So, how about creting a new 6.x-dev branch?
Tx.
Comment #33
ccprog commentedAlpha-2 works for me with D6.13.
Here is a patch that embraces #256795: Allow simplenews cron to send nodes that anonymous users cannot access. If you apply the patch published there to Simplenews, newsletters can be send with the user rights of the publisher of the newsletter instead of anonymous, which is what you get using Cron.
The same logic holds for Simplenews Scheduler. If you have nodes with restricted access inside your newsletter, they will only get out on scheduled issues if the user rights are changed at rendering time.
Comment #34
ccprog commentedOops, sorry, another change in my version got through that is not relevant here.
Comment #35
adam_b commentedAny chance of creating a dev release for those of us who don't play well with patches?
Comment #36
Sc0tt commentedI second that. Would be less intimidating for non-coders to start to download and test if it were listed as a 6x-dev module for download. Thanks for this module. I know a lot of people are waiting for this one. Who can I bribe to get this module completely ported faster?
Thanks very much,
Scott
Comment #37
sgabe commentedI sent an e-mail to dgtlmoon about a new 6.x-dev branch. I think this release is very alpha stage and much more people are needed to test it and report back to help the development. So keep an eye on this.
Comment #38
dgtlmoon commentedSorry for the delay, have just added the 6x dev branch, anyone want to take over some CVS work on this? i'll add you as a maintainer.. let me know, cheers.
Comment #39
sgabe commentedThanks for the 6.x-dev branch, I would help with pleasure with CVS if you add me as a maintainer to speed this up a little bit.
Comment #40
dgtlmoon commentedHey sgabe, it says "sgabe does not have a CVS account."
Comment #41
dgtlmoon commentedHave applied patches from comment #23 to latest 6x dev
Comment #42
dgtlmoon commentedOk well yesterday there was a 6x release on the project page but now it has gone away, i've rerun the CVS tag
cvs tag -b DRUPAL-6--1
5cvs update -r DRUPAL-6--1
but it looks like the tag is already there
i really hate CVS it's really getting in my way off getting any code out onto the drupal.org project, so..
@sgabe: are you confident in handling the 6x project stuff for this module?
Comment #43
sgabe commentedI took a look on the 6.x-dev branches, and I found both of them to be 5.x code so I committed the 6.x-1.0-alpha2 from #23 as 6.x-1.x-dev branch and applied the patch from #34. I think (if I am right...) with next cron run when drupal.org will update the download pages this will be fine and if you agree the 6.x-2.x-dev branch should be removed for now.
Comment #44
ccprog commentedPlease note that this means the scheduler will only work with the 6.x-1.x-dev version of simplenews, but not the official release!
Comment #45
sgabe commentedThis note is on the project page. The patch is not committed in the development snapshot (as far is I know) so I reworked it to be applicable on the current Simplenews 6.x-1.x-dev release.
Comment #46
sgabe commentedThe 6.x branch of Simplenews Scheduler is under active maintenance. The usage of the development snapshot is growing week by week, but very few people reporting back about bugs etc. Please, test it and report back, review patches, or create patches, your help is always welcome!
I am marking this as fixed, if anything comes up, that should be a new issue.