Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Here comes New features : 3 ways answers and not available case #22

Open
wants to merge 50 commits into
base: master
Choose a base branch
from

Conversation

BenaeSan
Copy link

Not available

Issue #20

a checkbox has been added to uncheck all the others checkbox, now you can show that you're not available for the meeting.

3 ways answers

Now you can create an advance poll, that give you the possibility to maybe be here for the meeting.

capture_1

capture_2

BenaeSan and others added 24 commits January 17, 2017 18:04
	attribut pollType
	+ traitement
	(WIP Non fonctionnel)
CHORE
	multiple value into slot selection
	create avdance answser poll
I18N
ADD
	multiple value into slot selection
	create avdance answser poll
I18N
FIX
	Simple Poll
I18N
FIX
	Simple Poll
I18N
	clean code
I18N
ADD
	not available checkbox whom pass to no all answer
ADD
	not available checkbox whom pass to no all answer Only for simple poll
	js to perform check on some change
ADD
	not avaible for advanced poll
ADD
	page with the days selection in a plain datepicker
	create days widthout slots
	Slot page + action
ADD
	page with the days selection in a plain datepicker
	create days widthout slots
	Slot page + action
	Copy the first line values to others
ADD
	select date page
	script in a external file
	script to select date and remove it
ADD
	delete selected date
FIX
	new Slot get current hour
	First columns width get readjusted
	clean code
I18N
	calendar not working due to double initilisation
CHORE
	clean code
I18N
	remove copyright header
FIX
	datepicker css impact on all site
ADD
	on change multi answer, the background color of cell change color to green/yellow/pink
ADD
	on change, change background color of the cell
	Clean code
ADD
	update days possibilty
ADD
	roundToQuarter function
CHORE
	del unused file
@juho-jaakkola
Copy link
Owner

Thanks for making the pull request!

Could you please change the indentations back to hard tabs (with width of 4 spaces)? The white space changes caused by them make it difficult to see what exactly has been changed.

bm jade added 2 commits February 8, 2017 12:02
ADD
	update poll
	update date
	delete slot
	Change space to tab
@BenaeSan
Copy link
Author

BenaeSan commented Feb 8, 2017

Yeah Sure, i've done and commited. Hope it that you want.

bm jade added 2 commits February 8, 2017 15:14
ADD
	error if no days' selected
FIX
	synch timestamp
bm jade and others added 20 commits February 9, 2017 10:28
FIX
	problem with time UTC comparation
CHORE
	format code
ADD
	answer object
ADD
	answer object
FIX
	Save the answer object
	number of columns in addSlot Page
	order of the answer response to show the undefined slot
FIX
	precheck case
FIX
	creation error if there is no vote value
CHORE
	clean code
FIX
	get the right vote value to update
CHORE
	clean code
FIX
	update date or days
FIX
	date format
ADD
	name in main form
FIX
	Counter for slot
ADD
	Number of Voters
ADD
	inner grid "pagination" repeat info all the tenth line
FIX
	get the previous polltype value on update
CHORE
	clean & comment code
# Conflicts:
#	actions/scheduling/answer.php
#	actions/scheduling/days.php
#	actions/scheduling/save.php
#	classes/ElggSchedulingPoll.php
#	classes/ElggSchedulingPollSlot.php
#	languages/fr.php
#	views/default/forms/scheduling/answer.php
#	views/default/forms/scheduling/days.php
FIX
	probleme affichage calendrier dans la V2
	Migrate page handler route
	move file from pages to resources
	change icon to font-awesome
FIX
	some undefined variable
I18N
	some forgotten label
	clean code
	clean code
FIX
	fix uninstanciate object in advance answers
	fix notification problem on create poll
@juho-jaakkola
Copy link
Owner

Apologies that it has taken so long to take a proper look at this. I will try to get to it soon.

@BenaeSan
Copy link
Author

No problem, really

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants