Skip to:
Content

BuddyPress.org

Opened 7 years ago

Closed 7 years ago

#7486 closed enhancement (fixed)

Improve message after successful group invitation acceptance.

Reported by: dcavins's profile dcavins Owned by: dcavins's profile dcavins
Milestone: 2.9 Priority: normal
Severity: normal Version: 1.0
Component: Groups Keywords: has-patch
Cc: dcavins

Description

Currently, when you accept a group invitation, you are returned to the invitations screen and the group you just joined is nowhere to be seen. You'd have to visit the "groups > memberships" screen to find a link to the group.

I propose that we add link to the group you just joined to the message that is displayed. Please see patch.

Attachments (2)

7486.1.diff​ (1.0 KB) - added by dcavins 7 years ago.
Improve "invite accepted" message.
accepted-invitation-message.png​ (26.7 KB) - added by dcavins 7 years ago.
Screenshot of improved message.

Download all attachments as: .zip

Change History (8)

@dcavins
7 years ago

Improve "invite accepted" message.

@dcavins
7 years ago

Screenshot of improved message.

#1 @hnla
7 years ago

Big +1 - Great enhancement.

#2 @dcavins
7 years ago

  • Owner set to dcavins
  • Status changed from new to accepted

This ticket was mentioned in ​Slack in #buddypress by hnla. ​View the logs.
7 years ago

#4 @r-a-y
7 years ago

  • Milestone changed from Awaiting Review to 2.9
  • Version changed from 2.8.2 to 1.0

Looks good to me!

One trivial thing is to add a space in the __() function call as per WP coding standards:
__( 'Group invite accepted. Visit %s.', 'buddypress' )

#5 @dcavins
7 years ago

@r-a-y Of course you're right. I'm going to hold off on committing this until we resolve #7485. This would be a handy place to use that new form of bp_get_group_permanlink(). :)

Thanks for your comments!

#6 @dcavins
7 years ago

  • Keywords dev-feedback removed
  • Resolution set to fixed
  • Status changed from accepted to closed

Resolved in r11529.

Note: See TracTickets for help on using tickets.