-
Notifications
You must be signed in to change notification settings - Fork 16
Initial Calendar Commit #26
base: master
Are you sure you want to change the base?
Conversation
Looks like the snapshot needs to be updated to get tests passing. Could you throw a screenshot in the main PR -- not necessary but would be nice :) Awesome work! Thanks |
@kcloud99 @russtaylor maybe we want to disable test snapshots? Do you guys normally use them? I know some people hate them and find them to be a pain and useless¿ |
A few other notes I should probably mention for future development: In this initial commit, A This Calendar module we're using can only be implemented through either |
Updated snapshot - or at least attempted to. |
Hrmm odd,, it failed, did you try running the command with |
This reverts commit a5269af.
@ctsstc yeah, I did that wrong. Reverting previous commit and running |
@ctsstc I think that did it. Thanks for the help there. |
@ctsstc I don't know about keeping the snapshots, I personally haven't used them before. It seems like they're more trouble than they're worth, but that's just on the surface. |
👍 |
This initial PR provides a basic react-based calendar that we can work with; allows for the inclusion of multiple event endpoints, is extensible with regard to stylings, and offers some basic tools we can begin working with.
Updating with screenshot per @ctsstc request
Also added
Link
inheader
component && updated styling for readability.Adds
moment
&&react-big-calendar
dependencies.