Skip to:
Content

BuddyPress.org

Opened 5 years ago

Closed 3 years ago

#8165 closed defect (bug) (fixed)

Email listing error when situation is not choose

Reported by: dhavalkasvala's profile dhavalkasvala Owned by: djpaul's profile DJPaul
Milestone: 10.0.0 Priority: normal
Severity: normal Version: 5.0.0
Component: Emails 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 5 years ago.

Download all attachments as: .zip

Change History (6)

@dhavalkasvala
5 years ago

#1 @dhavalkasvala
5 years ago

  • Keywords has-patch added

#2 @dhavalkasvala
5 years ago

  • Milestone changed from Awaiting Review to 6.0.0

#3 @imath
5 years ago

  • Component changed from Core to Emails
  • Milestone changed from 6.0.0 to 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
3 years ago

  • Keywords needs-refresh reporter-feedback added; has-patch removed
  • Milestone changed from Awaiting Review to 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 reproduce to reproduce the issue?

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

Version 0, edited 3 years ago by imath (next)

#5 @imath
3 years ago

  • Resolution set to fixed
  • Status changed from new to 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.