BuddyDev

Search

Replies

  • Participant
    Level: Master
    Posts: 279

    wilco Brajesh,

    Many thanks again!

    R.

  • Participant
    Level: Master
    Posts: 279

    hi Brajesh,

    thank YOU for fixing it!

    The inevitable question is: do you have an ETA for the next release, or should I use the github code directly?

    R.

  • Participant
    Level: Master
    Posts: 279

    hi Brajesh,

    using the code out of github, this looks like it’s working exactly as expected for single media uploads.

    I can also confirm that when I backout the github code, and use the WP code, the old behaviour returns.

    Looks very good!

    R.

  • Participant
    Level: Master
    Posts: 279

    hi Brajesh,

    thanks for your work on this. When I upload a new image, or comment on an existing image, the comments to *not* make it through to the single media page itself.

    When I comment on an existing activity thread, (on an image), this appears to work fine. I *think* this is Media->comment->feed works then in both directions. But media_in_feed->activity->[does_not_get_to_media].

    I did find something interesting. The comments appear to make it to the GALLERY just fine. Just *not* to the image itself. I’ve disabled my theme to make sure I’m not doing anything wonky, and the effect remains the same. No comments on the image.

    If that helps?

    R.

  • Participant
    Level: Master
    Posts: 279

    hi Brajesh,

    did you make progress on this, are there any signs of light on the horizon…?

    R.

  • Participant
    Level: Master
    Posts: 279

    hi Brajesh,

    I did some work with the BP activity comments tree. On it’s own it’s fine, but it is a different system from the WP comments tree, which is also fine on it’s own. Putting them together is fun!

    For different values of fun! 😉

    R.

  • Participant
    Level: Master
    Posts: 279

    hi Brajesh,

    ok, that’s a bit clunky (of buddyPress), and highly non-intuitive, but is certainly out of your hands.

    Many thanks for the clear explanation.

    R.

  • Participant
    Level: Master
    Posts: 279

    so what I mean is that if you fix the activity->comment->media issue, then I think we very nearly have a fully working system.

    The position of the activity in the stream is a side-show.

    R.

  • Participant
    Level: Master
    Posts: 279
    Richard Foley on in reply to: MediaModerator default Title suggestion #23080

    hi Brajesh,

    if I recall correctly, I had in mind something like a text field in the wp-admin/admin.php?page=bpmts-settings page for setting a default value, such that it ends up something like this:

    
    --- a/wp-content/plugins/media-moderator-for-mediapress/core/class-mpp-moderator-actions.php
    +++ b/wp-content/plugins/media-moderator-for-mediapress/core/class-mpp-moderator-actions.php
    @@ -62,7 +62,7 @@ class MPP_Moderator_Actions_Helper {
                    ?>
    
             <button class="mpp-moderation-report-btn" data-toggle="popover" data-title="<?php _e( 'Report Media', 'media-moderator-for-mediapress' ); ?><span
    -                data-content='<div class="mpp-moderator-report-form"><div class="mpp-moderator-message-info"></div><form class="mpp-moderator-form" method
    +                data-content='<div class="mpp-moderator-report-form"><div class="mpp-moderator-message-info"></div><form class="mpp-moderator-form" method
                     data-media-id="<?php echo $media->id; ?>"><?php _ex( 'Report', 'Label for report button', 'media-moderator-for-mediapress' ); ?></button>
    
    

    The relevant bit being placeholder="Enter Title" value="Inappropriate content"

    R.

  • Participant
    Level: Master
    Posts: 279

    hi Brajesh,

    oh, yes, interesting, I hadn’t seen the update. That’s *nearly* perfect.

    When I place the widget in my header, and in my footer both, I get:
    link(“Notifications (3)”)
    text(“You have 3 notifications”);

    and in the footer, I get:
    text(“You have 3 notifications”);

    What I’m after is:
    link(“You have 3 notifications”);

    If that makes sense?

    R.