@planner like I said, there are specific reasons the client wants the IP to be associated with each post. I can see either argument, but I can't agree it doesn't make sense. I do think it's valuable to have as much information tied to the post level as possible for forensic reasons.
Split JS Minification on Client/Board and ACP
It's a good idea, it would also reduce the size of nodebb.min.js
I just looked it up, there is no ACP related code in
nodebb.min.js, but it is also included on the ACP side for libs like
@psychobunny Plugins will grow. And adding JS related only for UX in ACP, isn't great
oh I see, you're looking to improve the ACP's performance. yeah that makes sense. I think all ACP needs is
Give ability to specify in
plugin.json, where my js should be concatenated
In my case, 95% of client JS is related to ACP, but It's part of
Gotcha, I totally forgot about the fact that
nodebb.min.jsincludes ACP plugin scripts as well. In that case I take my first comment back - even on this community it should cut down maybe 10-20kb worth
Nicolas, I'm assuming you don't want to put your js inline in the template?
For my plugins (I have my own fork of quickstart that I use), I actually make them require.js modules, so they are loaded on-demand: https://github.com/NodeBB/nodebb-plugin-quickstart/blob/julian/static/lib/admin.js
But looking back, this might also minified, so...