Drupal requires only a username and password by default.
In COD, we may need to add fields to support some of the features in other areas of the site.
For example, the Attendees list (/community) displays first name, last name, organization, job title, and interests. User profiles need to have those fields in order for this feature to be supported.
Another feature under discussion (http://drupal.org/node/1672922) is an opt-out feature for this list. Users should be able to opt-out and this data should be stored as a field in their profile.
Based on the features I'm familiar with, I propose the following as required fields during the user registration process:
1. Username
2. Password
3. Email address
4. Opt-in of all public lists (Boolean value)
In addition, the following fields should be optional:
4. Given name
5. Family name
6. Organization
7. Job Title
8. Interests
9. Photo
10. Accessibility requirements
11. Dietary requirements
12. Website or blog URL
13. Contact phone number
And some nice to have fields that allow for social media integration and funky displays but would require extra modules:
14. Twitter/Facebook/Google+ account
15. Geographic location
And finally, fields accessible to admins/organisers only, for notes. This field should not be visible to delegates. Ever.
16. Admin notes.
Once we decide the fields which will be used, this related issue will become a blocker. We still need agreement about a base system for user profiles: http://drupal.org/node/1597460
Please feel free to add any fields or other thoughts to this list!
| Comment | File | Size | Author |
|---|---|---|---|
| #51 | Screen Shot 2012-08-27 at 5.07.31 PM.png | 44.22 KB | saltednut |
| #50 | Screenshoot.PNG | 11.35 KB | mjonesdinero |
| #49 | 1673000-profile-fields-49.patch | 33.35 KB | dsdeiz |
| #46 | 1673000-profile-fields-46.patch | 9.47 KB | dsdeiz |
| #45 | dummy COD Dev Demo.png | 121.77 KB | dsdeiz |
Comments
Comment #1
sheldonkreger commentedAlthough many of these fields already exist in the user profile, we need to decide which will be asked for during the initial user registration process, as well. I believe all that is required now is a username and password.
The opportunities for prompting the user for this info are during initial site registration and during event registration. Many of these fields will never be filled in because users are never asked to fill them out.
Comment #2
twardnw commentedThey are asked for their First and Last names during account creation, nothing else at this time.
Comment #3
sheldonkreger commentedComment #4
twardnw commentedtagging
Comment #5
sheldonkreger commentedNote: I added an opt-in field for public display of name on the attendees list on this issue: http://drupal.org/node/1672922
Comment #6
cafuego commentedUpdated the summary, added email as mandatory (cannot communicate with delegate otherwise!) and added an optional field that I as a delegate usually expect (website) and ones that as conference organiser I would require (phone, accessibility, dietary).
(Cell/Mobile) phone is mostly important for presenters. It's valuable to be able to call them if they haven't turned up half an hour before they're scheduled to talk.
I also propose to rename 'first name' and 'last name' to 'given name' and 'family name' for easier cross-culture acceptance.
Comment #6.0
cafuego commentedUpdated issue summary.
Comment #7
cafuego commentedAttached is a patch that adds a few basic fields and sorts them into fieldgroups:
Admin: Administrative comments
Public: First name, surname, organisation, job title, website, picture
Private: Opt-In, dietary requirements, accessibility needs
I expect some might need to go into cod_base (eg: the admin one) and some into cod_community (the blog one) before we're done, but it's a start. This patch introduces a dependency on the link module.
This patch should apply happily *after*:
Comment #8
primerg commentedwhen i try to register or create a new account via the admin, i do not see the following fields:
6. Organization
7. Job Title
8. Interests
9. Photo
13. Contact phone number
When I edit, I do not see the following (even as an admin):
9. Photo
13. Contact phone number
Comment #9
cafuego commentedAh yes, they'll all be set as 'site admin' I reckon. Will reroll the patch after I get rid of that :-)
Comment #10
cafuego commentedRerolled patch with included permissions an a contact phone field.
This should be applied after:
#1477754-17: Rename "site administrator" role to "administrator"
#1672922-21: Recommended Changes to Attendee List (/community)
#1679940-8: Remove admin.module dependency
Comment #11
primerg commentedreviewing but found a problem with the dev im using. please see my comment and if anyone can answer http://drupal.org/node/1672922#comment-6262996my mistake!Comment #12
primerg commentedUsing the sandbox and applying the requested patched, I'm getting this error:
error: cod_community/cod_community.field_group.inc: No such file or directoryComment #13
primerg commentedcombining patch 7 and 10 and enabled photos
It's not clear in the discussion if we want to ask in the initial registration the following fields:
6. Organization
7. Job Title
8. Interests
9. Photo
Comment #14
sheldonkreger commentedPatch is clean against the current da_drupalcon sandbox. Form validation is thorough and all of the fields are working in both the admin area and for anon users creating their own account.
Let's leave the fields discussed in #13 out of the initial registration process. IMO, users prefer not to be asked too many questions upfront.
Comment #15
sheldonkreger commentedIt looks like the Feature for cod_community was displaying "needs review" but everything was default inside of it. I just rebuilt the Feature and re-rolled the patch in #13. Now everything is default again. So, this should be identical to #13. **grumble Features grumble grumble**
Comment #16
twardnw commentedThis patch applies clean, the fields are on the profile.
The one field I don't think should be there is 'dietary needs'. IMO this should be collected as part of the registration process, on the profile it implies that all events will have food ;)
Comment #17
sheldonkreger commentedI'm all over it . . . good call.
Comment #18
sheldonkreger commentedThis patch hides the meal preferences field from the registration form by default. cod_community Feature status is default.
Comment #19
twardnw commentedThink your patch missed something, that field was still showing for me. Try this one on.
Comment #20
sheldonkreger commented19 is clean and *successfully* removes the dietary needs field . . . and cod_community Feature is default. One more set of eyes here would be nice!
Comment #21
cafuego commented19 applied cleanly and the feature came up as defaults. The dietary requirements field is gone. The fields show up when they need to, screenshots attached.
12 suggests the fieldgroups I created never made it into my patch. So do we want to have fieldgroups for admin fields, potentially public fields and private fields? Or not bother?
I have moved the admin field to the top of the field list (not shown in the screenshots) as a conference administrator is more likely to want to see that first than the other fields. (Patch 21) This patch also removes the dependency on fieldgroup.
In theory the empty text on the attendee view could be changed, as there may be lots of signups, but they may all want to be anonymous. I don't think that's a major bug though :-)
Comment #22
primerg commented@cafuego, sorry, didn't mean to overwrite your work.
Regarding the public/private grouping, I think it will help if users know which info are publicly available and only for organizers. I personally would not want to post my phone number but I would be willing to give it to the organizers if they need to contact me.
Comment #23
cafuego commented's not your fault if I managed to make a patch that lacked a file :-)
I'll reroll it shortly.
Comment #24
cafuego commentedUpdated patch attached.
Comment #25
twardnw commentedPatch applies clean, love the new organization, makes things more clear. Thanks!
Comment #26
primerg commentedclean and it works as described
Comment #27
sheldonkreger commented24 is clean and has no annoying overrides!
Comment #28
twardnw commentedok, applied to sandbox
Comment #29
twardnw commentedcan we get:
?
Comment #30
cafuego commentedYes we can, but I think it would be really nice to use specific modules for those that provide API integration as well, so you're not just collecting a string in a text field.
I reckon that rather than adding extra dependencies on all the modules that provide that functionality on cod_community, it may be a good idea to add a cod_community_bonus (or something like that) feature that provides this integration with external social media services.
Comment #31
sheldonkreger commented#24 no longer applies after CODZILLA commit. This needs to be reworked.
As far as the fields in #29 are concerned, the only field which has a community module to handle it is Twitter. The rest will have to be text strings based on my perusing of related modules.
Comment #32
sheldonkreger commentedI'm working on this; all the changes should live inside cod_community, since we're collecting this data to feed the views created by cod_community.
Comment #33
sheldonkreger commentedHere's the patch.
I added the following fields to the user profile:
-Opt-in
-Given name
-Family name
-Accessibility requirements
-Dietary requirements
-Website/Blog URL
-Phone
-Location
-Twitter
-Facebook
-LinkedIn
-Google +
It also modifies the attendees view in cod_community to only list those who opt-in.
I added the dependency for field_groups, since there the user should know which data will be displayed publicly on their profile, and which should not.
None of the fields such as Facebook or Twitter are using any custom modules - just plain text. Trying to keep it simple.
There were some minor changes to cod_base because I re-arranged some fields in the user profile. Those are here as well.
Comment #34
twardnw commentedFamily Name and Given Name are just alternates for last and first (more global friendly)
Check the order of fields over ;) you've got first name, picture, last name...
Comment #35
sheldonkreger commentedOk I re-arranged the fields and removed the extra name fields.
Comment #36
twardnw commentedlost the field groupings, and the attendees list field
Comment #37
sheldonkreger commentedAlright, I'm hoping features has cooperated this time. I had to modify some of the files by hand.
Comment #38
twardnw commentedbig ole' PDO exception when I try and enable cod_base after applying your patch ;)
but you know this
Comment #39
cafuego commentedWill reroll this as fresh patch that should happily apply *after* "CODZILLA".
Comment #40
cafuego commentedReroll.
Location field uses the addressfield module (new depend).
I've slightly reworded the opt-in checkbox label.
Fields are back in their groups.
Applies to CODZILLA; patch created after applying #1279928: Features Extra support for date_formats to features (though no date formats are included, so that ought not matter).
Comment #41
cafuego commented[deleted double-submitted description without attached patch]
Comment #42
dsdeiz commentedWorks on my end. Not sure if I'm missing anything but I think there's a bit of inconsistency.
Comment #43
cafuego commentedYeah I was wondering about doing a twitter URL instead, but then I thought that if you want to put that field on a badge (generally more likely than the facebook, linkedin or G+ username) having a URL field for it is poo. I reckon as-is it can be themed to a link and still be helpful for badge data.
I'll check the grouping - I thought I'd put them near eachother :-/
Comment #44
cafuego commentedOdd. Attached is the layout on the site used to create the feature.
Also attached is an updated patch; I've visually inspected the field weights *and* added groups to the hidden fields. What I see now is in fields-admin-display.png.
Comment #45
dsdeiz commentedMaybe its somehow "random" because two elements have the same
#weight. dpm'ingfield_profile_facebookandsummary, I see the same value for#weightin both elements.Edit: Oh, #44 patch applies cleanly as well.
Comment #46
dsdeiz commentedI've changed the weight for "Facebook URL" up to "Twitter Username".
Comment #47
saltednut#46 applies cleanly but, when I enabled cod_base and cod_community I get a Notice and a PDOException:
Is there a patch I need to apply before using #46?
Comment #48
saltednutComment #49
dsdeiz commentedSorry, missed to add files.
Comment #50
mjonesdinero commented#49 patch applies cleanly and the urls are now below the member for:
attached is the image of the page that has the right display now
Comment #51
saltednut#49 applies cleanly. cod_community is default when enabled. I've tested this and it works showing the fields in a consistent order.
Comment #52
rootworkDon't want to derail this if it's really RTBC, but it'd be great to have a formatter that automatically linked the Twitter username to http[s]://twitter.com/...
I agree with making the Twitter username on its own rather than a link for badge and other purposes, but it would be nice to be able to click on it from the profile page anyway.
But if this is 99% done otherwise I don't want to stand in the way.
Comment #53
cafuego commentedLets leave it RTBC, get it committed and revisit it if needed/wanted.
Comment #54
japerryHi rootwork, lets take the extended fields (twitter) out of this feature. I'd actually like to put fields as something users would customize (and remove from cod_community eventually)
but for now I think this is a good upgrade path, and it needs to be committed. It tests properly from #49 to the latest git branch for cod and cod_support (Nov 5 2012) so pending Ezra's review, I think we're good to go.
Comment #55
rootworkI'm fine with that. Hooray for progress!
Comment #56
japerryOkay after talking to Ezra, we would like to refactor this into the commons profile module, which has many of the same fields. Marking as needs work until we get that integrated. It should just extend the fields that already exist for commons into this module instead of duplicating the common ones.
Comment #57
ezra-g commentedThe specific modules here are http://drupal.org/project/commons_profile_base and http://drupal.org/project/commons_profile_social.
A benefit of using those modules is that by aligning the profile fields between distributions, COD can get profile-related functionality built for Commons for free. For example, Facebook sign-in.
It also reduces code duplication between the projects.
Comment #58
rootworkJust curious if there had been progress on this at the recent sprints. Is integrating these two modules part of the near future, or just a "nice to have" goal for sometime ahead?
Comment #58.0
rootworkUpdated issue summary.