Skip to:
Content

BuddyPress.org

Opened 7 years ago

Closed 5 years ago

#8165 closed defect (bug) (fixed)

Email listing error when situation is not choose

Reported by: dhavalkasvala Owned by: DJPaul
Priority: normal Milestone: 10.0.0
Component: Emails Version: 5.0.0
Severity: normal Keywords: needs-refresh reporter-feedback
Cc:

Description

Email listing error when the situation is not to choose either there needs default value or any message this is required to choose. ​https://prnt.sc/pz6ypc

Attachments (1)

#8165.patch​ (1.2 KB ) - added by dhavalkasvala 7 years ago.

Download all attachments as: .zip

Change History (6)

@dhavalkasvala
7 years ago

#1 @dhavalkasvala
7 years ago

  • Keywords has-patch added

#2 @dhavalkasvala
7 years ago

  • Milestone Awaiting Review → 6.0.0

#3 @imath
7 years ago

  • Component Core → Emails
  • Milestone 6.0.0 → Awaiting Review
  • Owner set to DJPaul

Thanks for your patch, but choosing the milestone needs first a review from a member of the BuddyPress team. So I'll revert your last edit to this ticket.

#4 @imath
5 years ago

  • Keywords needs-refresh reporter-feedback added; has-patch removed
  • Milestone Awaiting Review → 10.0.0

Hi @dhavalkasvala

Let's have a look at this during 10.0.0. I'm not sure to fully understand your description for this ticket. Could you tell me the steps to do to reproduce the issue?

Also, could you improve your patch so that it follows ​WordPress PHP Code standards?

Last edited 5 years ago by imath (previous) (diff)

#5 @imath
5 years ago

  • Resolution → fixed
  • Status new → closed

In 13186:

Make sure a situation is attached to a BP Email before displaying it

Props dhavalkasvala

Fixes #8165

Note: See TracTickets for help on using tickets.