Skip to:
Content

Opened 6 weeks ago

Last modified 3 days ago

#7897 assigned defect (bug)

Nouveau - Can't post on Activity Directory page

Reported by: r-a-y Owned by: imath
Milestone: Awaiting Review Priority: normal
Severity: normal Version: 3.0.0
Component: Templates Keywords: needs-testing
Cc:

Description

Reported here:
https://buddypress.org/support/topic/cant-post-in-activity-stream/

User cannot post on the Activity Directory page (also doesn't load correctly), but can on their own profile's Activity page.

Screenshots:

http://smiling.wpengine.com/wp-content/uploads/2018/06/Screen-Shot-2018-06-07-at-2.28.06-PM.png
http://smiling.wpengine.com/wp-content/uploads/2018/06/Screen-Shot-2018-06-07-at-2.28.12-PM.png
http://smiling.wpengine.com/wp-content/uploads/2018/06/Screen-Shot-2018-06-07-at-2.28.21-PM.png

Attachments (1)

7897.patch (632 bytes) - added by imath 6 weeks ago.

Download all attachments as: .zip

Change History (8)

#1 @imath
6 weeks ago

  • Keywords reporter-feedback added

Hi r-a-y

It’s probably a JavaScript issue, can we have a screenshot of the user’s browser inspector ?

« Divi » is the name of the theme ?

How can we reproduce this ?

#2 @imath
6 weeks ago

Just tested in TwentySeventeen, and it works as expected.

https://cldup.com/Ji24rW01Jy.png

#3 @r-a-y
6 weeks ago

  • Keywords reporter-feedback removed

Here's another thread with similar problems with a screenshot of the browser log:
https://buddypress.org/support/topic/some-activity-stream-issues/

Screenshot:
http://dl3.joxi.net/drive/2018/06/06/0017/2626/1178178/78/0a3b06d22d.jpg

#4 @imath
6 weeks ago

  • Milestone changed from Awaiting Review to 3.2.0
  • Owner set to imath
  • Status changed from new to assigned

Thanks @r-a-y. It looks like it's a post-form.php template override. Problem is this template override is using a Legacy like markup. Nouveau is injecting Backbone views for the Post Form. I'll work on a patch to disable the JavaScript in this case.

@imath
6 weeks ago

#5 @imath
6 weeks ago

Well, no in case of a template override there's no problem actually 🤔

The patch is adding a check to be sure JS templates are available. Still a mystery...

#6 @DJPaul
5 weeks ago

Making sure that the templates we rely on are actually present is a good practice, and if it doesn't solve exactly this problem, it would be an improvement, as I'm sure it will fix edge cases.

We probably need to work on messaging around Nouveau that if you had customised Legacy templates, you're going to have problems.

#7 @r-a-y
3 days ago

  • Keywords needs-patch removed
  • Milestone changed from 3.2.0 to Awaiting Review

Moving back to Awaiting Review as we haven't been able to verify the report.

Note: See TracTickets for help on using tickets.