Really awesome plugin. I'm excited to see the gif format being replaced by webm slowly but surely.
[nodebb-plugin-emoji-extended] Emoji Extended (reworked)
PS: An introduction on creating new sets can be found here.
Now this is just plain cool...
However I'm not sure in which way NodeBB should/could keep emoji-extended within its dependencies.
If emoji-extended is listed (and activated by default), some unused client-side code gets delivered.
If emoji-extended and non-static set(s) are listed (and activated by default), that/those set(s) would still need to update files by hand via emoji-extended ACP page.
The options that come to my mind:
emoji-extendedand a static set (
emoji-cubicoppis the only one right now); add that set to default settings of
emoji-extendedtherefor it is activated by default.
emoji-extendedand a non-static set (
emoji-onewould be my choice); add that to default settings; call its
./nodebb setup; I could add a callback-based variant for that set if you want to avoid promises within core.
emoji-extendedand a non-static set; add that to default settings; call its
update()function on server start (
static:app.preloadI assume) if no files are present.
emoji-extended; do not enable it by default
- drop emoji from dependencies
The 3rd would break my clear distinction between plugin installation and image files o_O .
I tend to the 5th; Adding the word "smiley" to the description of emoji-extended should ease the search within the plugin-list.
It would be great if NBBPM could handle
nodebb-[...]-properly (or does it already?) for this case.
@frissdiegurke No way, the last time I added a
peerDependenciesproperty to my
package.json, things went sideways really really quickly.
It's deprecated in npm3, and I'm not touching that property with a ten-meter pole
@julian deprecated? you sure? the docs only specify a behavior change that those are not installed automatically
Perhaps you're right -- in any case, I was initially drawn to
peerDependenciesbecause it seems like a good way to handle sideways dependent packages, but it didn't work at all in practice, so I've abandoned its usage.
@julian Since npm
peerDependenciesother than outputting warings, you might consider giving it a try again :D
Otherwise (to keep stuff working with
2-) we could add a field like
Dependency management is definitely not something I want to get into
Perhaps do a
module.parent.require('nodebb-plugin-emoji-extended')and error out if it returns null?
@PitaJ But only as long all version ranges match a single version, if they don't it communicates with a "silent" nested
nodebb-plugin-emoji-extendedwhich would probably cause much confusion and thus I don't want this ;) The main problem is that NodeBB at some point breaks the distance to NPM by depending on the
node_modulesstructure (of which I cannot come up with an alternative way of course). This makes it impossible to solve the issue with NPM only. It has to be either a
peerDependencywhich will not install the package if it misses, but does not cause other possible trouble or a solution via NodeBB itself. Since NodeBB breaks the distance to NPM it should be responsible for that breakpoint. At least these are my thoughts of it.
dependencywould also disrespect the
nodebb-plugin-emoji-extendedwhich might cause even more trouble.
I think as long NodeBB does not provide a better way, going with
peerDependencyand a notice within the plugins descriptions is the best I can do.
rod last edited by
nodebb-plugin-emoji-appleI see in my log this error:
26/2 10:35  - error: [plugins/emoji] nodebb-plugin-emoji-extended is not installed. nodebb-plugin-emoji-apple depends on it.
I do have
nodebb-plugin-emoji-extendedinstalled. To make sure nothing was wrong with the base emoji-extended plugin i removed it and the newly installed emoji-apple plugin, reloaded NodeBB, and re-installed but still receive this warning message.
It seems that the plugin is working though but feel that I should report this as something maybe wrong that I haven't found yet.
@rod I assume you have any
1.0.0installed. I should precise that message o_O
FYIL: to install any
^1.0.0version you currently have to remove it from the package.json file ;)
And if this is the case it's not using the
nodebb-plugin-emoji-appleat all :P
rod last edited by
@rod If you're on master you can install it without any problems, just need to remove that listing within package.json (you'll have to
git reset --hardbefore future
git pullto avoid problems with
If you're on another branch (e.g.
0.9.x) it's probably not working to its full potential. For example I think you'd have to update the composer too in order to get auto-completion...
Here's an interesting bit of behavior :P
- Upgraded to 0.9.4.
- Removed dependency from package.json
- NPM i [email protected]
- Installed nodebb-plugin-emoji-apple
Are the emoji working within posts/previews? If not, check your config.json for the
I'm not sure why the translations of
modal.legalare not working, will take a look at that.
@frissdiegurke It looks like the URL was malformed and after correcting it, emojis are back up and running.
@ThingBreaker And as to the translations it seems the
Since it's a minor
0.9.xissue and I'm supporting
^1.0.0only, it's not gonna be fixed from my side. Maybe the @administrators want to merge the fix of
0.9.x, but I wouldn't bet on this ;) . I guess it's something one can live with until next major NodeBB release.
ngadmin last edited by ngadmin
Does it work with proxied nodebb? I tried to use it but the images are called with url http://localhost:4567 :(