Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Here's the use case:
A site is in private beta. "invites only" are allowed to register. Existing users should be able to send invites to people, so they can register immediately, WITHOUT including the relationship request. It would just be a simple checkbox for each possible relationship: "Send 'Friend' request too?"
That way, they can still tell people how cool the site is, and you should totally register, but you're not necessarily going to create a user relationship with my account.
Make sense?
Comment | File | Size | Author |
---|---|---|---|
#8 | ur_invite_request-1430920-6.patch | 1.81 KB | Shawn DeArmond |
#2 | ur_invite_request-1430920-2.patch | 1.78 KB | Shawn DeArmond |
#1 | ur_invite_action-1430926-2.patch | 1.31 KB | Shawn DeArmond |
Comments
Comment #1
Shawn DeArmond CreditAttribution: Shawn DeArmond commentedThis patch is better. I didn't need to check for UR-UI, since it's a dependency anyway.Oops wrong issue. Meant to put this patch on #1430926: When "The following recipient is already a member:", provide link for relationship request
Comment #2
Shawn DeArmond CreditAttribution: Shawn DeArmond commentedOkay, here's the patch for this issue. It creates a checkbox that uses form #states to show the radio buttons for relationships (if applicable). If it's unchecked, it's a normal invite without a UR request. If it's checked, it also includes the UR request.
Since UR-Invite currently has a bug, until it gets committed, you'll have to first apply patch #1 on #1345694: UR-Invite doesn't create required relationship before applying this patch.
Comment #4
Shawn DeArmond CreditAttribution: Shawn DeArmond commentedOf course it failed testing, since it needs that other patch first.
Comment #5
Berdir#2: ur_invite_request-1430920-2.patch queued for re-testing.
Comment #6
BerdirThe comment should be on a separate line and start with an uppercase letter.
Otherwise, happy to commit this, sounds useful.
Comment #7
Shawn DeArmond CreditAttribution: Shawn DeArmond commentedPatch updated to fix the comment.
Here's what it looks like now:
Comment #8
Shawn DeArmond CreditAttribution: Shawn DeArmond commentedLet's attach the patch this time.
Comment #9
BerdirCommited, thanks for the re-roll.