Skip to content
This repository has been archived by the owner on Dec 10, 2024. It is now read-only.

Fixing bug where timestamp is expecting a numeric value #33

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

dotstormz
Copy link

No description provided.

@michaelyudin
Copy link

Can you please explain the bug you are having? Full Calendar should be passing a timestamp already. This PR does not work on my end. Are you using a different version of FullCalendar?

@JulienPradet
Copy link

It's not a timestamp anymore with a newer version of fullcalendar.
Source: http://fullcalendar.io/docs/event_data/events_json_feed/

Will this eventually be merged ?

@Toasterson
Copy link

There is another pull request which updates fullcalendar to 2.0.2 and fixes the Controller Accordingly.
See #24. You could also use my fork if you want a newer version of fullcalendar.

@JulienPradet
Copy link

Thanks. Your fork works like a charm indeed. :)

@mikeyudin
Copy link
Member

I will merge Toasterson's #24 and report back. Please give me some time to get this merged.

@coveralls
Copy link

Coverage Status

Changes Unknown when pulling df77ae2 on dotstormz:master into ** on adesigns:master**.

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

Successfully merging this pull request may close these issues.

6 participants