Skip to content

Conversation

jsma
Copy link
Contributor

@jsma jsma commented Oct 29, 2014

NOTE This is not ready to merge yet but want to get feedback on this work-in-progress.

Added a stub for editing basics doc that is focused on non-technical audiences.
Added to the 'how to contribute' to explain how to set up local environment for testing documentation changes.

Added a stub for editing basics doc that is focused on non-technical audiences.
Added to the 'how to contribute' to explain how to set up local environment for testing documentation changes.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We use (or should) British English for django CMS docs.

@evildmp
Copy link
Contributor

evildmp commented Oct 29, 2014

Can we have a chat about this on the #django-cms IRC channel (freenode.net)? It'll be easier to do it there, for the high-level view.

@coveralls
Copy link

coveralls commented Oct 29, 2014

Coverage Status

Coverage decreased (-0.1%) to 90.704% when pulling 2b24ed2 on jsma:documentation into 2fa7c67 on divio:develop.

@digi604
Copy link
Contributor

digi604 commented Nov 3, 2014

lgtm

@evildmp
Copy link
Contributor

evildmp commented Nov 4, 2014

Hi John. #3557 incorporates your changes, in a new structure (with the user-oriented material in the section user).

@jsma
Copy link
Contributor Author

jsma commented Nov 4, 2014

Thanks! I'll take a look and give feedback when I get a chance, has been a very busy past few days but I do have loads of screenshots ready to go for the end-user documentation. Should I work off your branch then or will these changes be merged soon?

@evildmp
Copy link
Contributor

evildmp commented Nov 4, 2014

I think it would make sense if we both worked off this branch for now, and made sure we kept each other up-to-date with progress. But, I would like to get this merged sooner rather than later.

@jsma jsma closed this Nov 10, 2014
@jsma jsma deleted the documentation branch November 10, 2014 17:51
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.

4 participants