Closed jez closed 9 years ago
@Z1MM32M4N how's this going?
I've been stuck on how to get this to work with Rail's custom form builders. I'd like to be able to just write f.datetime
and have it include all the necessary HTML. For whatever reason, when I try to modify the existing FormBuilderWithDateInput
class, it raises a method not found exception.
Good idea. I think changing FormBuilders requires a server restart, so you know.
@dlbucci Rather, when I try to add the field builder: FormBuilderWithDateInput
to the appropriate form (without changing the form builder) is when I get errors.
What's the method it's not finding? Are you modeling the form after the assessments#edit form (which might be the only place we use that builder)?
pm'ed you with a lengthy explanation.
So turns out that a server restart did the trick (as referenced in the above pull request!).
The next issue is more of a design decision, so I'm wondering what people's thoughts are.
There's a project that serves the assets through the Rails asset pipeline, but it's out of date right now (1 version behind).
We could very easily download the required assets (Bootstrap 3 DatePicker + Moment.js) and manage installation and updating manually, or we could load the assets through our Gemfile. I'm in favor of the second, but there's no guarantee that this will happen any time soon.
Thoughts? Our options are:
I'd say just do it manually (at least for now). I think we shouldn't be dependent on another party for this sort of simple stuff.
I made an initial stab at upgrading the datetime pickers. You can see my progress in #226. I strongly suggest breaking it down by the individual commits, as I removed a lot of cruft as a part of the process.
Hey @jez, your PR #255 sort of broke check boxes in forms, or at least the check boxes for modules in the assessment edit form. Each box is now the checkbox, followed by "Has module", a new line, then "module".
Good catch, I'll take a look when I wake up. On Thu, Feb 5, 2015 at 11:40 PM Daniel Bucci notifications@github.com wrote:
Hey @jez https://github.com/jez, your PR #255 https://github.com/autolab/Autolab/pull/255 sort of broke check boxes in forms, or at least the check boxes for modules in the assessment edit form. Each box is now the checkbox, followed by "Has module", a new line, then "module".
— Reply to this email directly or view it on GitHub https://github.com/autolab/Autolab/issues/181#issuecomment-73182026.
@jez reminder (i know you've been busy over the weekend)
Can this be closed now?
It hasn't hit the master branch yet On Wed, Feb 11, 2015 at 11:43 AM Daniel Bucci notifications@github.com wrote:
Can this be closed now?
— Reply to this email directly or view it on GitHub https://github.com/autolab/Autolab/issues/181#issuecomment-73971053.
Required Features
This will likely be implemented using Bootstrap 3 Datepicker, which includes support for all of these date and time options.