#3336 closed defect (bug) (fixed)
Themes: Use Cavalcade for Trac sync
Reported by: | obenland | Owned by: | obenland |
---|---|---|---|
Milestone: | Priority: | normal | |
Component: | Theme Directory | Keywords: | has-patch dev-feedback |
Cc: |
Description
Ever since the Theme Directory was updated two or three years ago, the script that syncs theme review results over to the directory has been running as a cronjob on my sandbox.
With the introduction of Cavalcade on w.org, this is no longer necessary and it should be moved over.
Attachments (1)
Change History (8)
#4
@
7 years ago
Overall it looks fine to me.
- I'd probably reduce
theme_directory_check_cronjobs
to every 15m, it's not as crucial as on the plugin directory where there's many events per minute that need managing / catching if they fail. - Rather than
require_once WP_PLUGIN_DIR . '/theme-directory/lib/class-trac.php'
I'd userequire_once dirname( __DIR__ ) . '/lib/class-trac.php';
Note: See
TracTickets for help on using
tickets.
Agreed. How often does the cron job run?