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

Craft3 #8

Merged
merged 3 commits into from
Sep 4, 2018
Merged

Craft3 #8

merged 3 commits into from
Sep 4, 2018

Conversation

jcdarwin
Copy link

Upgrade of the UserManual plugin to work in Craft 3.

Functionality and attribution have been kept as per the Craft 2 version.
There was no licensing specified with the original plugin that I could find, so this branch specifies MIT and credits @RobErskine, but this can be changed if necessary.

I'll be using this Craft3 version from my fork in my projects, but look forward to being able to switch to using this repository once it's merged.

Given that many users will still want access to the original Craft 2 version, you might want to merge this craft3 branch as a separate branch (i.e. still keep it named craft3), or merge it onto master, but subsequently include a note in the README about how users can access the Craft 2 version (i.e. via a git tag / commit hash).

Any comments welcome.

@jcdarwin jcdarwin mentioned this pull request Aug 25, 2018
@RobErskine
Copy link
Owner

@jcdarwin !!!

This is awesome, thanks so much for your hard work. Let us give this a once over in the AM and we'll merge tomorrow.

@RobErskine RobErskine merged commit f0711e8 into RobErskine:master Sep 4, 2018
@jcdarwin jcdarwin deleted the craft3 branch September 6, 2018 10:21
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