This is a minor, minor thing, but it seems like the settings menu item should have a description. E.g.:
$items['admin/settings/feedapi'] = array(
'title' => 'FeedAPI settings',
'description' => "Basic settings for FeedAPI module.",
'page callback' => 'drupal_get_form',
'page arguments' => array('feedapi_admin_settings'),
'access arguments' => array('administer feedapi'),
);
Happy to offer a patch if you want one.
| Comment | File | Size | Author |
|---|---|---|---|
| #4 | description-settings.patch | 473 bytes | crookednumber |
Comments
Comment #1
alex_b commentedGlad to accept one :)
Comment #2
crookednumber commentedAbsolutely.
The language OK?
Love the module, BTW. I'm getting a ton of use out of it.
Comment #3
alex_b commentedLanguage could be more specific. What about:
"Control allowed tags and allocate cron time for FeedAPI."
?
Comment #4
crookednumber commentedPatch attached.
Comment #5
aron novakThank you for the patch, however that item has description already:
http://cvs.drupal.org/viewvc.py/drupal/contributions/modules/feedapi/fee...
Do we want to change the current text: "Configure advanced options for FeedAPI module." ?
Comment #6
crookednumber commentedSorry. I totally missed that. I was looking at HEAD.
Comment #7
aron novakWell, in fact FeedAPI misuses HEAD, so it's not your fault at all.
Comment #8
crookednumber commentedThanks. I'm still getting my head around CVS tags and branches and I thought this was another 2 steps back in my education. Phew!
Comment #9
aron novakI think this is settled now.