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

Time limited polls #17

Open
powli opened this issue Dec 12, 2016 · 11 comments
Open

Time limited polls #17

powli opened this issue Dec 12, 2016 · 11 comments
Assignees
Milestone

Comments

@powli
Copy link
Member

powli commented Dec 12, 2016

Add a date option to limit a poll to a set date. After the specified day is finished, no further votes are allowed and the poll should only be visible in result mode.

Things to do:

  • add date picker to poll creation form
  • add end date to model
  • change rendering in poll list view to indicate finished polls (either by marking them or by seperating all finished polls from the non-finished)
@powli powli added this to the 0.1 milestone Dec 12, 2016
@powli powli mentioned this issue Dec 15, 2016
@powli powli assigned powli and longtomjr and unassigned powli Dec 19, 2016
@longtomjr
Copy link
Collaborator

see #22
Date time picker was added, and the model has been modified to contain the end date.

What still needs to be done:

  • Render the poll different in the poll list
  • Have an indication of when the poll wil close and/or a countdown.
  • Not allow voting on a closed poll

@longtomjr
Copy link
Collaborator

Note, you are currently bale to create time limits in the past, this should not be allowed.

@powli
Copy link
Member Author

powli commented Jan 9, 2017

Quick thought:
We should also determine if we want to save a timestamp or just the date (and end the poll on the next day GMT-wise).

@longtomjr
Copy link
Collaborator

Currently we are saving the time and date in UTC, I think having time suport is helpfull

@longtomjr
Copy link
Collaborator

longtomjr commented Jan 11, 2017

Disallowed the possibility to select a date in the past with a minimum of 2 hours length. This is not checked on post yet.

TODO:

  • Validate the post request
  • Render the poll different in the poll list
  • Have an indication of when the poll wil close and/or a countdown.
  • Not allow voting on a closed poll.

What needs to be determined:

  • Whether or not we should have the ability to pick time, or just support dates.

Pings for thoughts: @powli @Tranberry @NogginBops @Dormanil @vogonistic

Feel free to edit the message as the state of this issue change.

@NogginBops
Copy link

I like the ability to set time limits, but that could be problematic with regards to different time zones. Shouldn't be too hard to implement though I imagine.

@longtomjr
Copy link
Collaborator

Implementation is not a problem. It is more about the direction we want to go with this. I like the ability to set the time, but there is a case to be made to have the time set only to the nearest hour, to simplify picking a date and time.

@NogginBops
Copy link

I like the nearest hour idea, maybe like increments of half an hour could work?

@powli
Copy link
Member Author

powli commented Jan 12, 2017

I'm also just for the nearest hour for now. If it seems necessary we can implement more detailed times later anyway.

For timezones, I would just add a select field with all time zones to the poll creation and normalize them to GMT upon saving to the DB. So the server can just compare GMT timestamps internally.

We might need to change the timestamps in the frontend according to the timezone of the user, but I would consider that future jazz.

@BraedonWooding
Copy link
Member

Should the creator of the poll be able to change the date during the time it's open? Or could that be abused to do undemocratic actions?

@longtomjr
Copy link
Collaborator

@powli as far as I am aware moment() (what is used by the bootstrap widget) automatically works in the user's timezone and will convert it to GMT on post.

I dont think we should have the ability to change the time the poll closes once it is set. It allows for system abuse, and for the poll to remain democratic, the rules need to be stated from the start (afaik).

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

No branches or pull requests

4 participants