Page 16 of 18

Re: [3.1][BETA] Advanced Polls 1.2

Posted: Sun Apr 23, 2017 7:12 pm
by Mohammed AlSarsawi
I have project that need a daily performance reports
so the pall contain some tasks and the users must update there performance every day
and users must don't put more than 1 point or vote for each task in a single day -for example-

depending on kind of tasks
this reports not needed any more next month or week so we reset the votes and start again
but it's difficult to manually reset 10-20 polls or more every time

I think this 2 features is important for all performance based projects and companys

Re: [3.1][BETA] Advanced Polls 1.2

Posted: Sat Apr 29, 2017 8:55 pm
by njs
A quick bug fix.

The message details button doesn't show the text in 3.2, somehow there's something forcing the font-size to be 0 in buttons.css.

Add this to advancedpolls.css and it will show up again:

Code: Select all

.post-buttons .button span {
    font-size: 13px !important;
}

Re: [3.1][BETA] Advanced Polls 1.2

Posted: Sat Apr 29, 2017 10:05 pm
by javiexin
Thanks for the report. Have you checked it is a core issue, or is it related to other extension?
To test it, disable all other extensions, refresh the browser cache, possibly even use a different browser if available, and check it does the same. If it does, it is a core 3.2 change that is affecting the extension. If it works in this environment, then it is a extension compatibility issue.

Anyhow, I have not yet released a 3.2 ready version of this extension. If you use it in 3.2, I would be glad to get your feedback. You have to "remove" (disable, not use, etc.) the Poll Ended Notification feature, as that is definitely 3.1 only. The rest should mostly work. There might be some issues with the styling and javascript, due to changes in the prosilver style for 3.2 (quite significant), but other than than, should be quite ok.

Again, thanks for the report.
-javiexin

Re: [3.1][BETA] Advanced Polls 1.2

Posted: Sun Apr 30, 2017 7:05 am
by njs
I'm not sure this is 3.2 only or not as I only tested in 3.2 and not on 3.1, but prosilver's buttons.css has this entry:

Code: Select all

.post-buttons .button span {
	font-size: 0;
}
That's why there's the need to fix it in your css.

Re: [3.1][BETA] Advanced Polls 1.2

Posted: Sun Apr 30, 2017 10:10 pm
by Chabniest
javiexin wrote:
Sat Apr 29, 2017 10:05 pm
Thanks for the report. Have you checked it is a core issue, or is it related to other extension?
To test it, disable all other extensions, refresh the browser cache, possibly even use a different browser if available, and check it does the same. If it does, it is a core 3.2 change that is affecting the extension. If it works in this environment, then it is a extension compatibility issue.

Anyhow, I have not yet released a 3.2 ready version of this extension. If you use it in 3.2, I would be glad to get your feedback. You have to "remove" (disable, not use, etc.) the Poll Ended Notification feature, as that is definitely 3.1 only. The rest should mostly work. There might be some issues with the styling and javascript, due to changes in the prosilver style for 3.2 (quite significant), but other than than, should be quite ok.

Again, thanks for the report.
-javiexin
Hi,

I am testing your extension with phpBB 3.2 and only disable the Poll Ended Notification feature doesn't resolve the "Set Notification" error message.

Re: [3.1][BETA] Advanced Polls 1.2

Posted: Sun Apr 30, 2017 11:21 pm
by javiexin
Please, try the following:
Disable the extension. If possible, delete data, to be sure.
Edit the config/services.yml file in the extension, and remove the highligthed lines:
https://github.com/Wolfsblvt/advancedpo ... ml#L23-L52
Now, enable the extension, and test it. Of course, do NOT activate poll end notifications.
Let me know how it goes.
Thanks,
-javiexin

Re: [3.1][BETA] Advanced Polls 1.2

Posted: Mon May 01, 2017 8:39 am
by Chabniest
this worked great.

Re: [3.1][BETA] Advanced Polls 1.2

Posted: Mon May 01, 2017 12:18 pm
by javiexin
Glad it worked for you. Any further issue with 3.2, please, report here.

Regards,
-javiexin

Re: [3.1][BETA] Advanced Polls 1.2

Posted: Fri May 05, 2017 11:42 pm
by Neverlands
Great I've found the solution here :D
I had exactly the same problem with the notifications options.

Re: [3.1][BETA] Advanced Polls 1.2

Posted: Mon May 15, 2017 1:04 pm
by njs
Any plans for supporting notifications in 3.2 ?

Thanks for the extension, it's really useful.

Re: [3.1][BETA] Advanced Polls 1.2

Posted: Mon May 15, 2017 1:28 pm
by javiexin
njs wrote:
Mon May 15, 2017 1:04 pm
Any plans for supporting notifications in 3.2 ?

Thanks for the extension, it's really useful.
Yes, but not any time soon. I have other priorities unfortunately.

Re: [3.1][BETA] Advanced Polls 1.2

Posted: Wed May 17, 2017 7:13 pm
by njs
While creating a poll on my board I got the following error:

https://s18.postimg.org/984genqkp/Erro.jpg

It seems that the poll_start is getting a negative value. Is this already known ? I didn't set any strange dates or so.

Thanks

Re: [3.1][BETA] Advanced Polls 1.2

Posted: Wed May 17, 2017 7:47 pm
by javiexin
njs wrote:
Wed May 17, 2017 7:13 pm
While creating a poll on my board I got the following error:

https://s18.postimg.org/984genqkp/Erro.jpg

It seems that the poll_start is getting a negative value. Is this already known ? I didn't set any strange dates or so.
Known bug. Solution here: viewtopic.php?p=14571996#p14571996

-javiexin

Re: [3.1][BETA] Advanced Polls 1.2

Posted: Wed May 17, 2017 9:23 pm
by njs
Thanks :) I didn't go too far back looking for possible solutions, sorry.

Re: [3.1][BETA] Advanced Polls 1.2

Posted: Fri May 19, 2017 7:26 pm
by Bruce Banner
Hi. There's something confusing me. There's an option with this extension to show poll voters, but there's also a permission in the forum permissions section of the acp to allow people to see poll voters. So if a group has permission to see poll voters but this extension's option to show poll voters is set to no, does that mean that that group won't be able to see them despite the permission?

EDIT: Oh wait, is that permission in the acp part of this extension?