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

remove events list beneath calendar #45

Open
wants to merge 30 commits into
base: develop
Choose a base branch
from

Conversation

mattwr18
Copy link
Collaborator

up test coverage
closes #44

Kachulio1 and others added 28 commits October 12, 2018 17:15
- add logic to route to the current path when mounting
- remove the active attr from Dropdown to fix the error in the console
* Update README.md

* Update README.md

* add event calendar to events page

* Show events in calendar

* events showing up in calendar

* use spread operator for events

* style calandar month view

* set up local development proxy
…entures-website-react-front-end into 13-events-calendar
…site-react-front-end into 44-remove-events-list-beneath-calendar
@ghost ghost assigned mattwr18 Dec 18, 2018
@ghost ghost added the Needs Review label Dec 18, 2018
@coveralls
Copy link

coveralls commented Dec 18, 2018

Coverage Status

Coverage increased (+72.2%) to 100.0% when pulling 0691e40 on 44-remove-events-list-beneath-calendar into 1c5e938 on develop.

@mattwr18
Copy link
Collaborator Author

Not testing line 20 of the EventsList... I found this discussing that steered me away from trying to mock the fetchEvents function in this test reduxjs/react-redux#325

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

Successfully merging this pull request may close these issues.

Remove events list from beneath the calendar
5 participants