[nodebb-plugin-ns-awards] NS Awards, Give Medals!
-
@torn2 Just finished my exams, so I'll have time to continue this.
I think that with the last commit i've got it, there are just some other things to do i.e. beautify the ACP panel, the all awards page and also to add more rules, but the core thing is there. -
It looks like, ability to set achievements should not be a part of the Awards plugin.
In order to track several behaviors for every user in the system, plenty of additional database objects should be created. It's an overhead. Only who is interested in such resource overhead, only they should spend extra CPU/RAM resources for it.What does it mean?
- NS Awards plugin should provide an API
- New Achievement plugin should be introduced, where Achievement plugin can interact with NS Awards if needed.
-
-
You can make it like a daily cron that checks award conditions instead of checking all conditions all time.
-
Would love, love, love to see automated medals that function similarly to rewards, where it awards medals based on parameters.
-
I am testing
v1.4.0and have problems with this plugin. The award images are not being displayed in either the profile of the user or the topic that an awarded user starts.
The images do show up in the ACP for the plugin:
Is this a known problem with this plugin and the recent version of NodeBB?
-
@rod said in [nodebb-plugin-ns-awards] NS Awards, Give Medals!:
@rod I believe I found the problem. The path to the image is missing a dividing slash.
.../uploads/awards/award-...
Odd thing is that I cannot seem to figure out where to add the missing
/in the plugin code. -
I assume It was changed in NodeBB. That is why It does not work anymore for you in the latest version.
-
@Nicolas Your broken plugin was bothering me! I found a work around.
In my public directory I created a symbolic link to the proper directory with the broken directory path. Now the award images render properly.
This is what I did:
In the
publicdirectory I created the symlinkln -s uploads/awards uploadsawards -
Sorry for that.
In some particular version of the NodeBB, trailing slash was removed from the path.
With the next update, the path will be handled better. -
@Nicolas I'm not sure if you were still planning on doing an update, but I've made this plugin work with 1.4.5: https://github.com/NicolasSiver/nodebb-plugin-ns-awards/pull/13
It still needs some work on the ACP side, and you'd probably want to check out one of the hotfixes I made before merging (see issue).
I think you should check out NodeBB's
src/file.jsinstead of writing your own uploader though
Thanks! -
I had a feeling that it's not a good idea to do changes in the
master
I have started to work on the next major update, so a lot of parts of the plugin is not implemented or unfinished. Anyway, thanks for the pull-request. -
@MBFTOM I understand what you are asking, but
NS Awardsplugin was designed with a goal to give special awards. I mean awards when you don't mind to do it not so often and you are eager to write a reasonI'm exploring an idea of external API for the plugin, so it will be possible to write another plugin which can collect a lot of data about users behaviors and grant awards. Since I think auto-behavior should be decoupled from Awards. What you are asking is an extra load on the forum to capture activity and store (more noise in your storage) a result of the capture.
-
I look forward to making the things which in the list. https://github.com/NicolasSiver/nodebb-plugin-ns-awards#todo
-
@Nicolas said in [nodebb-plugin-ns-awards] NS Awards, Give Medals!:
I'm going to eliminate all items from TODO list with v4 release. It will be big.
Waiting Impatiently!
