If this is your real code, then you're missing the return $items; line at the end, looks good otherwise :)

P.S.: permission strings are usually all lowercase (they are a technical identifier, not a user facing label or so).

If this is your real code, then you're missing the return $items; line at the end, looks good otherwise :)

Edit: You are using 'type' => MENU_SUGGESTED_ITEM which means that the menu item needs to be enabled manually in the administration. And when you do, it will show as a sub menu item of /user/%user, which basically means it is invisible because there the default secondary menu position only displays top-level menu items. Try MENU_LOCAL_TASK instead to have it show up as a local task.

P.S.: permission strings are usually all lowercase (they are a technical identifier, not a user facing label or so).