Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Custom Profile Updates #302

Merged
merged 11 commits into from Apr 5, 2013
Merged

Custom Profile Updates #302

merged 11 commits into from Apr 5, 2013

Conversation

Spuds
Copy link
Contributor

@Spuds Spuds commented Apr 5, 2013

Adds the ability to have a custom profile field show on the memberlist. Mostly a follow on to the removal of the built in IM items, this allows new ones to be defined an shown.

Some minor refract of the function to move the querys to subs

Should fix the use of the superglobals at some point

Spuds added 11 commits April 4, 2013 11:25
Signed-off-by: Spuds <spuds@spudsdesign.com>
Signed-off-by: Spuds <spuds@spudsdesign.com>
…mberlist.subs.php

Signed-off-by: Spuds <spuds@spudsdesign.com>
Signed-off-by: Spuds <spuds@spudsdesign.com>
…m fields

Signed-off-by: Spuds <spuds@spudsdesign.com>
Signed-off-by: Spuds <spuds@spudsdesign.com>
Signed-off-by: Spuds <spuds@spudsdesign.com>
Signed-off-by: Spuds <spuds@spudsdesign.com>
Signed-off-by: Spuds <spuds@spudsdesign.com>
Signed-off-by: Spuds <spuds@spudsdesign.com>
Signed-off-by: Spuds <spuds@spudsdesign.com>
norv added a commit that referenced this pull request Apr 5, 2013
@norv norv merged commit cb7eb9c into elkarte:master Apr 5, 2013
@norv
Copy link
Contributor

norv commented Apr 5, 2013

Thanks! I'm merging this as it is, I'm thinking that we can tweak or improve more as another step.

@Spuds
Copy link
Contributor Author

Spuds commented Apr 5, 2013

Thanks ... I'll take a look at moving things to a new table ... need to make a commit to update the installer anyway as this requires a new col in the custom_fields table (that I forgot to place in this PR )

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants