User Profiles & Custom Profile Fields (again)

[wpillar]

Pretty happy with this now, here's a summary:

Added

• User profile page at /user/<slug>.<id>
• Custom profile fields in UCP and displayed on User profile.
• Form\Renderer class that can take a Form\RenderableInterface and generate the HTML for it. Input, select, or textarea. This is used by the custom profile fields to dynamically create form elements.
• form_render_field() twig function that takes a Form\RenderableInterface and uses the Form\Renderer.
• UserController currently used for the public facing user profile.
• profile_fields table.
• profile_field_options table.
• user_profile_fields table.

Future Work

• Currently the slug for the user profile isn't used at all, not sure for what (if anything) I should be using it for? I guess we need to generate the link to a profile for a given user with their slug?
• ACP interface for adding custom profile fields.
• Extending custom profile fields with functionality for displaying the fields in sign up, running the fields through the parser.
• Adding the Message, Follow & Buddy functionality to the User profile.
• Unit tests.

Looking For

Looking for this PR to be reviewed with a view to merge, aim here is a stable MVP of the features described with scope for future work to be conducted separately.

[euantorano]

I'll have a look over this today. In regards to the slug, it should just be the user's name, but slugged (using Laravel's Str::slug(), so My Username becomes my-username). It isn't used for lookups, just to provide "pretty" looking URLs :smile:

So when a user registers or changes username, their slug should be updated, and then just passed whenever building the URL to the profile.

[euantorano]

Other than my nitpicks, this looks good :)

[wpillar]

@euantorano think we're all good, anything else blocking a merge?

[euantorano]

I don't believe so. I'm on mobile right now, but my last look through looked good :)

On 31 Mar 2015, at 20:50, Will Pillar notifications@github.com wrote:

@euantorano think we're all good, anything else blocking a merge?

— Reply to this email directly or view it on GitHub.

[wpillar]

Great! :D if you could merge it when you get a chance, that'd be swell.