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

Add routes and marker functionality #10

Merged
merged 2 commits into from
Dec 26, 2017
Merged

Conversation

TwoWheelDev
Copy link
Contributor

I have added a routes shortcode, to enable displaying a route from Strava in the same way activities are displayed.

I have also added an option to display markers at the start and finish points of the route or activity. This defaults to false to preserve backwards compatibility fro users upgrading their plugin

@cmanon cmanon requested a review from jrfoell December 18, 2017 13:42
@jrfoell
Copy link
Collaborator

jrfoell commented Dec 26, 2017

@DLintott I know we don't always strictly follow WP coding standards, but I'm trying to convert to it (gradually). I'm going to do some minor formatting on your PR before merging it.

Most of all - THANK YOU!!!

@jrfoell jrfoell merged commit abbdf7a into cmanon:master Dec 26, 2017
@TwoWheelDev TwoWheelDev deleted the routes branch December 26, 2017 22:02
@TwoWheelDev
Copy link
Contributor Author

@jrfoell Cheers! Haven't really done much WP plugin coding before, so haven't really looked at the standards before.. so just tried to stick with what was already there.

@jrfoell
Copy link
Collaborator

jrfoell commented Dec 26, 2017

@DLintott do you have a WP.org profile name that you use for development or support? I'd love to give you a shout-out and I can also add you to our plugin contributors list 👍

@TwoWheelDev
Copy link
Contributor Author

@jrfoell I didn't have one.... but I do now! Same name as on here.... dlintott 👍

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.

2 participants