Open columbian-chris opened 2 years ago
Howdy Chris, Thank you for this suggestion. This is a great idea.
We are considering revamping the plugin data system to reduce metadata bloat as referenced in another issue. However, this small fix could help reduce the bloat in the short term.
Feel free to fork and modify the plugin for your own use or to open a PR and we will review it. As an open-source SDK company, OneSignal appreciates feedback and suggestions from the community.
Thanks again
Thanks for the response. I did open a pull request: #287 👍
I have noticed that in the editor screen all of my custom post types that the OneSignal meta box is displaying. Could we throw in a hook after line 258 of
onesignal-admin.php
(v2.2.4) to be able to edit the post types that the meta box will appear on? I don't need the meta box for any of my post types except for "post" and it would be strongly preferred to not have it display on all the additional post types to avoid bloating my meta data database table with unnecessary details from that meta box and to prevent even the possibility for anyone with Wordpress admin access to be able to submit a notification push.So here's the current line 258-260 of
onesignal-admin.php
:Add a filter hook in to be able to have control of which post types to show the meta box on in the editor:
And then in let's say for example my theme's
functions.php
file, I could add this: