[nodebb-plugin-poll] Poll plugin
-
@Gordon-Gu said:
@yariplus said:
gist:8fbed0308828c980e117
GitHub Gist: instantly share code, notes, and snippets.
Gist (gist.github.com)
no luck:(
my poll.json
The result
Ah okay, I should have posted my whole code. Looks like the client doesn't load the language files automatically, you have to you have to call load() first in the poll init and translate() on the template. If you can wait a bit I'm going to make a PR with everything setup.
edit: hmm, actually it does load the language automatically when you call translate(), maybe someone who knows more about the translator knows why it's not translating that template. All the admin templates work fine for me cause their parsed on the server.
edit 2: Okay! I got it working. PR coming...
-
GitHub - yariplus/nodebb-plugin-poll at translator
Poll plugin for NodeBB. Contribute to yariplus/nodebb-plugin-poll development by creating an account on GitHub.
GitHub (github.com)
Comes with Russian translations.
-
-
@Leonardo-Galani Just gave this a shot on 0.7.3 and it appears to be broken on my end.
Edit: It appears to work but it doesn't display the poll on the topic.
-
@Kowlin There is a pull request (not authored by me) which fixes the problem. The Javascript problem is easy to fix. However, that in and of itself does not wholly fix the problem. The second part is that the Persona theme is laid out a bit different in HTML and the poll client javascript expects to insert the poll into a certain HTML element. Altering the poll code to work correctly for Persona means breaking all other themes. I have my own fixes for these issues but haven't pushed anything because of the theme compatibility problem.
How I fixed the JS problem:
index 5eb6a09..7b78147 100644 --- a/public/js/poll/main.js +++ b/public/js/poll/main.js @@ -1,7 +1,7 @@ (function() { window.Poll = { load: function(data) { - Poll.sockets.emit.load(data.pollid, function(err, poll) + Poll.sockets.emit.load(data, function(err, poll) { if (!err) { Poll.view.init(poll, function(pollView) if (parseInt(poll.info.deleted, diff --git a/templates/poll/notify.tpl b/templates/poll/notify.tpl index a4c5354..fc693c2 100644 --- a/templates/poll/notify.tpl +++ b/templates/poll/notify.tpl @@ -1 +1 @@ -<script>Poll.load({ pollid: {pollid} })</script> \ No newline at end of file +<script>Poll.load({pollid})</script>And how I fixed the Persona theme problem:
diff --git a/public/js/poll/view.js b/public/js/poll/view.js index 1d586ad..900f3e4 100644 --- a/public/js/poll/view.js +++ b/public/js/poll/view.js @@ -63,7 +63,7 @@ }, insertPoll: function(poll, callback) { View.parsePoll(poll, function(html) { - $('#post-container .post-row[data-index="0"] .po + $('.posts .clearfix[data-index="0"] .content').p callback(); }); }, -
@Schamper This plugin would be very usefull in fact. I'm pretty sure that some people are moving from NodeBB to Discourse because of this problem. About me I spent some time installing nodeBB, which is fabulous, but the BIG problem is that I didn't find any working poll system on NodeBB until now, and I'm thinking about migrating to Discourse.. Sad, don't you think ?
-
lmao @Schamper being the reason why you are giving up on forum software.
I think people need to understand that he has done those plugins as side projects that are open source. He has no reason to even update these other than his own motivation.
Yes it would be nice for them to work on the latest version but if you are that desperate for a plug then use the nodebb version that the plugin supports.BitBangers is still running nodebb v0.6 as we rely heavily on the shoutbox which is only really compatible up to v0.6.
-
@Schamper The poll plugin is now working on my 0.8.2 NodeBB. Thank you very much for this awesome work !!
-
Just wanted to let everyone know that this plugin is still being worked on. A couple weeks back I spent a lot of time reworking the server code and yesterday I spent some time reworking the poll creator. Some notable new features:
- The creator will look for an existing poll in your post and allow you to edit it
- Adding options is a bit more obvious now
- The date-time picker now works again
- The preview of your post doesn't show some bastardised list anymore
- Lots and lots of code improvement
Next up is working on the poll viewer and some other stuff. Hopefully I'll be able to release a 0.9.x compatible version soon
-
@whitts currently it's still a design decision to only allow polls in the first post. With the new hooks (well, new since when this was originally created) however, it should be possible to allow a poll by editing the first post.
Maybe with poll 2.0
I'd need some good arguments to why allowing a poll in every post will be a good idea. -
-
if possible creator of thread put more than one poll in thread and maximum votes per user i will move to nodebb from mybb.
something that i need much, but mybb not want offer for me so http://community.mybb.com/thread-154224.html
-
@martec max votes per user on a single poll? That's already supported. Multiple polls per thread is not something I'm considering, I'd much rather go with multiple questions on a single poll. Either way, it'll be something that will take quite some effort to implement, given the current method of doing polls. I do want to have something like that supported some day, though.
-
hi, thanks for answer.
what the difference with multiple questions on a single poll and Multiple polls per thread?
what i need that thread creator create multiple poll, so probably in this case not has any difference?maybe, will complicate when need change one of the question, why if need add one option in one of the questions, probably need recreate all questions. If plugin accept more than one poll, will need recreate only question that need made change.
anyway, i will wait more progress before migration to nodebb.
