search

amgoodfellow / reactjs-calendar

A new version of the react calendar. One that uses webpack 2, and is made with ADA compliance in mind
Apache License 2.0
3 stars 3 forks source link

Addresses #49 #50

Closed amgoodfellow closed 7 years ago

amgoodfellow commented 7 years ago

Closes #49

kdjohnson commented 7 years ago

Also I get this error Unknown prop "contentProps" on <div> tag. Remove this prop from the element. in MonthView when I paginate to the beginning of the term.

kdjohnson commented 7 years ago

I get this addComponentAsRefTo(...): Only a ReactOwner can have refs. You might be adding a ref to a component that was not created inside a component'srendermethod, or you have multiple copies of React loaded when I change to the schedule view

amgoodfellow commented 7 years ago

Nice catch on the logging, and I fixed the contentProp warning. That third one I'm not able to duplicate. When exactly do you get it?

kdjohnson commented 7 years ago

As soon as I change to the schedule view

amgoodfellow commented 7 years ago

To update, for posterity, KaJuan was testing the calendar in his courses portlet and found this issue. The error is being caused by a ref attribute on a Paper component. Also, because of this, I realized the calendar is needlessly causing courses to pull in React as a dependency twice. The next push will delete the ref and add react and react-dom as peer dependencies.