It's missing right now - you can still access the settings/config page at admin/config/fullcalendar, but there's no way to get there from the admin interface.

CommentFileSizeAuthor
#7 fullcalendar-1033110-7.patch1.58 KBtim.plunkett
#2 fullcalendar-1033110-1.patch568 bytestim.plunkett
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

geerlingguy’s picture

geerlingguy CreditAttribution: geerlingguy commented
Title: Add link to config page on Drupal "Configuration" page » Add link to config page on Drupal "Configuration" and Module overview pages

Should also appear on the module overview page, next to the Permissions link.

tim.plunkett’s picture

tim.plunkett
he/him
Primary language English
Location Philadelphia
CreditAttribution: tim.plunkett commented
Status: Active » Needs review
FileSize
fullcalendar-1033110-1.patch568 bytes

Ugh, wish this had made it in for 7.x-1.2

geerlingguy’s picture

geerlingguy CreditAttribution: geerlingguy commented
Status: Needs review » Needs work

Still doesn't appear on the admin/configure page, though... but sounds like you can fix this by assigning it to an existing category on that page (according to your note in IRC). This patch makes the Configuration link appear correctly on the module overview page, though.

tim.plunkett’s picture

tim.plunkett
he/him
Primary language English
Location Philadelphia
CreditAttribution: tim.plunkett commented

Which category to assign it to?
I thought "Content Authoring", but the consensus on IRC was "Regional and Language".

geerlingguy’s picture

geerlingguy CreditAttribution: geerlingguy commented

Content Authoring makes more conceptual sense to me... has nothing to do with geography or translation...

tim.plunkett’s picture

tim.plunkett
he/him
Primary language English
Location Philadelphia
CreditAttribution: tim.plunkett commented

Yes its not related to regions, but its more related to date&time than authoring content. Ugh.

tim.plunkett’s picture

tim.plunkett
he/him
Primary language English
Location Philadelphia
CreditAttribution: tim.plunkett commented
Status: Needs work » Needs review
FileSize
fullcalendar-1033110-7.patch1.58 KB

Hmm, how about adding our own category?

geerlingguy’s picture

geerlingguy CreditAttribution: geerlingguy commented
Status: Needs review » Reviewed & tested by the community

I like it, and I think this would be fine... and it works fine.

Only caveat - I wonder if we should hold off on that until we collaborate with the Calendar module maintainers (and possibly other event-related modules) to make sure everyone is using this menu item. If multiple modules define that menu item, will there be any problem?

aspilicious’s picture

aspilicious CreditAttribution: aspilicious commented

I agree there isn't a good category for this, it's sad the block gets added on the bottom left, but hey it's working and we need those admin links :)

tim.plunkett’s picture

tim.plunkett
he/him
Primary language English
Location Philadelphia
CreditAttribution: tim.plunkett commented
Status: Reviewed & tested by the community » Fixed

If other modules redeclare admin/config/calendar, it won't affect us.
The only "problem" is if they declare something like admin/config/calendars, because that would be silly.

Neither iCal nor Calendar have admin settings.

http://drupal.org/cvs?commit=485810

aspilicious’s picture

aspilicious CreditAttribution: aspilicious commented
Status: Fixed » Closed (fixed)

Cleanup of the issue queue!