Output a message in bio.install pointing user to settings page.

Sorry, new link:

http://codebaboon.com/usability-suggestion-display-settings-path

Good idea. However:

A) this doesn't follow our coding standards (no tabs -- we use 2 spaces).
B) that's not the right way to handle t() and a link, since then it's backwards in RTL languages. You want something like:
t('Blah blah: <a href="@url">link text</a>', array('@url' => url('/path/to/link')));
C) we don't generally print raw URL paths like that in help messages -- either just make the "bio settings page" itself a link, or use the funky "Administer >> User Settings >> Bio settings" stuff that we use in help texts.

Thanks,
-Derek

Sure, thanks for the pointers. Here's a new patch, with the line:

drupal_set_message(t('You can configure the bio module on the <a href="@url">bio settings page</a>',array('@url' => url('admin/user/bio'))));

A few more minor nits:
D) Help messages like this should end with a period.
E) Our coding style wants a space after the comma here: page</a>',array('@url' like so: page</a>', array('@url'.
F) Don't forget to set issues back to "code needs review" when you upload a new patch to address the concerns when someone marked it "code needs work". ;)

That said, the changes were so minor, it's not really worth the time to go back around again waiting for you to make a 2 character change to the patch and resubmit it. ;) I just fixed these myself and committed to the DRUPAL-5 branch.

Thanks for the usability improvement.

Great, sorry for wasting your time, but thanks for pointing out the errors!

Automatically closed -- issue fixed for two weeks with no activity.