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

Fix for future events #14

Merged
merged 3 commits into from
Feb 19, 2018
Merged

Conversation

Dean-Coakley
Copy link
Collaborator

@Dean-Coakley Dean-Coakley commented Feb 19, 2018

Fixes: #13

I've made an absolute mess of version control/commit messages. Would you like me to fix them first?

All html changes are just linting. Which I can revert if you don't like it.
Only code changes are in main.js

@davej davej merged commit 9564806 into davej:master Feb 19, 2018
@davej
Copy link
Owner

davej commented Feb 19, 2018

Thanks Dean. Nah that's fine. I'll Merge.

If you're interested in adding CI linting to the repo. Then I've connected the repo with travis-ci. If it's something you're interested in doing and want help then just create an issue.

Thanks, catch you on Thursday!

@Dean-Coakley Dean-Coakley deleted the future-event-patch branch February 19, 2018 15:31
@Dean-Coakley Dean-Coakley mentioned this pull request Mar 3, 2018
Dean-Coakley added a commit to Dean-Coakley/CorkJS that referenced this pull request May 12, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Upcoming event is incorrect
2 participants