[nodebb-plugin-poll] Poll plugin
-
I have
<a class="poll-view-result-votecount" href="#"> "[[poll:votes]]"</a>(without the quotes)
-
err, it keeps parsing out the text
This is what i'm using
https://gist.github.com/yariplus/8fbed0308828c980e117 -
@Gordon-Gu said:
@yariplus said:
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...
-
https://github.com/yariplus/nodebb-plugin-poll/tree/translator
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. -
