|
Geek Gazette GAZ is a community newsletter management system. |
Advertisement | |
Learn how to remove ads |
|
Thread Tools | Display Modes |
#1
|
|||
|
|||
GAZ 1.0.0 beta 1 feedback and suggestions from Milad
Installation process:
- As of vB 3.6 you can add the cron job in the XML product file, I didn't like the fact that I had to setup a cron job manually while that can be done via XML file. readme.html file: correct the path: http://www.yoursite.com/forums/geek/..._send_cron.php to http://www.yoursite.com/forums/geek/...n/gaz_send.php Settings: - I found my personal email in Reply address setting, and this must be the default one of the forum $vbulletin->options['webmasteremail'] - Style and user information (Style, Userid and User Name): Here it's better to use the own setting for each user, and it's better to send the newsletter from the default email $vbulletin->options['webmasteremail'] not from the admin membership. Newsletters: - It's better to determine the scheduled time for each newsletter while adding or editing it not via the cron job which have to run to decide if it's the time to generate an issue or not. (I figured out later, this can be set in generators section) - timing could be daily, weekly, biweekly and monthly .. etc. Issues: - Management here is good. Generator: - Management here is good. Found errors: - In Generators section, I chosen Preview last HTML issue, then new window popped up with this url http://www.syrianmeds.net/forum/.php?id=1&prev=1 - Also the link for Plain Text preview is http://www.syrianmeds.net/forum/?settings[scriptname].php?id=1&txt=1&prev=1 - The previous problems disappeared in newly generated issues, I think it's default data problem. - In modules section, I receive this warning: Warning: Missing argument 5 for run_mod() in /geek/gaz/includes/gaz_class_core.php on line 226 - In Plain Text preview, there are HTML code. - In sending section for an issue I had this error Warning: Missing argument 1 for gaz_class_cron() in /geek/gaz/includes/gaz_class_cron.php on line 22 - While you add a module to module set, if you set an order, it won't be set, you have to right it in order field and save again. Sent HTML issue: - It must have the HTML direction is wrong, it's ltr while my forum is rtl - I choose not to show replies and views but their fields are still exist as empty fields. - some parts don't obey right-to-left standards. Notes: - templates have to be grouped in the admincp |
#2
|
|||
|
|||
Quote:
Also, it might be a good idea to offer 'Use User Style' as an option, although this will cause more processing. Quote:
Quote:
|
#3
|
|||||||
|
|||||||
Thanks for taking the time to write this up Milad, it really helps!
Quote:
There is no other way around this (other than sending the emails out manually which I have made pretty automated too). Quote:
Quote:
Quote:
Quote:
Quote:
Quote:
Thanks again for your feedback. It helps loads |
#4
|
|||
|
|||
Quote:
PHP Code:
Quote:
Quote:
Can you add an option to use the actual header and footer and navbar in the newsletters? I'm waiting for beta 2 Thanks |
#5
|
||||
|
||||
Hey Milad,
The problem isnt in finding a place with vB's cron job - it is the way that vBs heduled task system works. vb's heduled task is not a real cron - it runs off the back end of another user. There is a web site that you can schedule tasks on for free. Basically you register the URL and time with the site and it calls the script at that time. That would be the best way forward. Maybe Kirb has another idea. It was much more complex than that. I too assumed that would be the case, but it was a challenge to load up a mock user, language and style via a cron and via a normal page. The interim solution is a setting. I wont be adding the option for the regular header and footer simply because it will consist of sending javascript files and functions too. I suspect there could be serious problems with that - maybe Kirby can comment? |
#6
|
|||
|
|||
I see no real problem with running a heavy processing script through vB Cron Jobs. They are triggered via a 1 Pixel Image, so they don't really affect the browsing user triggering execution.
Things to keep in mind though: It might be necesary ton increase max_execution time and ignore user abort. Also, if processing takes > 1 minute (which is probably the case) it might be necessary to build the next execution time before the end of script execution, so it does not run into race conditions. |
#7
|
|||
|
|||
I assure I understand how vB's crons work, via that 1 pixel image which is cron.php?cronid=XX. me too, don't see a real problem with running a heavy processing script through vB Cron Jobs.
Best regards |
#8
|
|||
|
|||
Quote:
What is about automated dropping process? |
#9
|
||||
|
||||
The GAZ header and navbar use most of the default header. The navbar is slightly different as it doesn't contain any javascript menus, nor irrelevant items. Customising it is pretty straight forward and simple.
As for the cron job - I have never seen anything apart from caution from vb staff about its cron jobs being used for massive tasks - hence my hesitation from utilising vBs scheduled task system for this purpose. TBH it is easier to set it up as a cron job (it originally was) so I assume that it will get folded back in. |
#10
|
|||
|
|||
I browse to the file gaz_send.php and I get this repeated error:
Warning: Missing argument 1 for gaz_class_cron() in /geek/gaz/includes/gaz_class_cron.php on line 22 Warning: Missing argument 1 for gaz_class_cron() in /geek/gaz/includes/gaz_class_cron.php on line 22 Warning: Missing argument 1 for gaz_class_cron() in /geek/gaz/includes/gaz_class_cron.php on line 22 Warning: Missing argument 1 for gaz_class_cron() in /geek/gaz/includes/gaz_class_cron.php on line 22 Warning: Missing argument 1 for gaz_class_cron() in /geek/gaz/includes/gaz_class_cron.php on line 22 Warning: Missing argument 1 for gaz_class_cron() in /geek/gaz/includes/gaz_class_cron.php on line 22 Warning: Missing argument 1 for gaz_class_cron() in /geek/gaz/includes/gaz_class_cron.php on line 22 Warning: Missing argument 1 for gaz_class_cron() in /geek/gaz/includes/gaz_class_cron.php on line 22 Warning: Missing argument 1 for gaz_class_cron() in /geek/gaz/includes/gaz_class_cron.php on line 22 |
Thread Tools | |
Display Modes | |
|
|
Similar Threads | ||||
Thread | Thread Starter | Forum | Replies | Last Post |
GAZ Private Beta 3 Release Discussion | The Geek | Geek Gazette | 12 | 04-21-2007 01:22 AM |
GAZ Private Beta 2 Release Discussion | The Geek | Geek Gazette | 5 | 04-11-2007 12:40 AM |
feedback | kidmercury | Geek Article and Review System | 0 | 11-01-2005 04:01 AM |