Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

no email notification, sub-post only text link #1

Open
benjino opened this issue Jun 18, 2013 · 3 comments
Open

no email notification, sub-post only text link #1

benjino opened this issue Jun 18, 2013 · 3 comments

Comments

@benjino
Copy link

benjino commented Jun 18, 2013

First of all, I don't know Git so I installed the zipped version. I installed the new dev version of bp moderation and used it. It now works in the sense that the flag/unflag now does something. It will flag and unflag as you click on the link, and it will show in BP Moderation > Contents admin area.

So, I flagged all the test activity posts I could and looked at the BP Moderation > Contents, and it was all showing. And, I also unflagged them all and then logged back into the BP Moderation > Contents to find nothing there. This is good.

One thing though. The flag/unflag icon you created only shows in the first post. It will not show in responses or sub-posts under an initial post, only an underlined text link (flag/unflag) shows. It would be nice to have the flag/unflag icon in all activity posts.

I did not create any queries yet, not sure how to even use that. I just tested out of the box functionality.

Thanks for getting back to this plugin!

Keep sending me updates to your work on this!

Ben

@francescolaffi
Copy link
Owner

thanks for the feedback, I'll look into it.

@francescolaffi
Copy link
Owner

as for the icon in the comments, i left it without flag for consistency with the other links in that position, but if you want the flag you can do with a filter:
when this is run apply_filters('bp_moderation_get_link', $link, $type, $is_author, $id, $id2); and $type is activity_comment you can remove with string replace the class bpm-no-images from $link, before returning it.

@benjino
Copy link
Author

benjino commented Jul 8, 2013

I added a flag with CSS using fontawesome, it works the same with the CSS structure I have. Thanks though.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants