Enhance docs #291

Merged
merged 35 commits into from Dec 9, 2016

Conversation

Projects
None yet
7 participants
@datakurre
Member

datakurre commented Sep 15, 2016

This pull request is for reviewing, fixing and merging docs written during Mephisto sprint

Would fix #253

@AnneGilles

This comment has been minimized.

Show comment
Hide comment
@AnneGilles

AnneGilles Sep 15, 2016

Contributor

I just added that other email address to settings... and now?

Contributor

AnneGilles commented Sep 15, 2016

I just added that other email address to settings... and now?

@gforcada

This comment has been minimized.

Show comment
Hide comment
@gforcada

gforcada Sep 15, 2016

Contributor

@AnneGilles as soon as you make a change or close and re-open a pull request mr.roboto (the thing behind does checks) runs again.

Contributor

gforcada commented Sep 15, 2016

@AnneGilles as soon as you make a change or close and re-open a pull request mr.roboto (the thing behind does checks) runs again.

@gforcada

This comment has been minimized.

Show comment
Hide comment
@gforcada

gforcada Sep 16, 2016

Contributor

@datakurre @plone/documentation-team where is this going to be uploaded?

Contributor

gforcada commented Sep 16, 2016

@datakurre @plone/documentation-team where is this going to be uploaded?

@datakurre

This comment has been minimized.

Show comment
Hide comment
@datakurre

datakurre Sep 16, 2016

Member

Currently Mosaic builds at Travis and uploads to S3. The link is in README.

On 16. syyskuuta 2016 klo 11.26 +0300, Gil Forcada Codinachs notifications@github.com, wrote:

@datakurre (https://github.com/datakurre) @plone/documentation-team (https://github.com/orgs/plone/teams/documentation-team) where is this going to be uploaded?


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub (#291 (comment)), or mute the thread (https://github.com/notifications/unsubscribe-auth/AAJyvzo9-uagyNc6-M7qEI9R4mFVBGYwks5qqlKYgaJpZM4J-IsG).

Member

datakurre commented Sep 16, 2016

Currently Mosaic builds at Travis and uploads to S3. The link is in README.

On 16. syyskuuta 2016 klo 11.26 +0300, Gil Forcada Codinachs notifications@github.com, wrote:

@datakurre (https://github.com/datakurre) @plone/documentation-team (https://github.com/orgs/plone/teams/documentation-team) where is this going to be uploaded?


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub (#291 (comment)), or mute the thread (https://github.com/notifications/unsubscribe-auth/AAJyvzo9-uagyNc6-M7qEI9R4mFVBGYwks5qqlKYgaJpZM4J-IsG).

@AnneGilles

This comment has been minimized.

Show comment
Hide comment
@AnneGilles

AnneGilles Sep 16, 2016

Contributor

once the docs have been polished they should go to readthedocs imho. that would make for a nicer URL.

good to see it here now, though:

http://plone-app-mosaic.s3-website-us-east-1.amazonaws.com/latest/getting-started.html

Contributor

AnneGilles commented Sep 16, 2016

once the docs have been polished they should go to readthedocs imho. that would make for a nicer URL.

good to see it here now, though:

http://plone-app-mosaic.s3-website-us-east-1.amazonaws.com/latest/getting-started.html

@datakurre

This comment has been minimized.

Show comment
Hide comment
@datakurre

datakurre Sep 16, 2016

Member

Unfortunately we cannot autogenerate screenshots at RTD, but we should be able to integrate with docs.plone.org.

On 16. syyskuuta 2016 klo 11.45 +0300, AnneGilles notifications@github.com, wrote:

once the docs have been polished they should go to readthedocs imho. that would make for a nicer URL.

good to see it here now, though:

http://plone-app-mosaic.s3-website-us-east-1.amazonaws.com/latest/getting-started.html


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub (#291 (comment)), or mute the thread (https://github.com/notifications/unsubscribe-auth/AAJyv9ki3nwNIsCPoWoLN00fdGPgQfUAks5qqlcwgaJpZM4J-IsG).

Member

datakurre commented Sep 16, 2016

Unfortunately we cannot autogenerate screenshots at RTD, but we should be able to integrate with docs.plone.org.

On 16. syyskuuta 2016 klo 11.45 +0300, AnneGilles notifications@github.com, wrote:

once the docs have been polished they should go to readthedocs imho. that would make for a nicer URL.

good to see it here now, though:

http://plone-app-mosaic.s3-website-us-east-1.amazonaws.com/latest/getting-started.html


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub (#291 (comment)), or mute the thread (https://github.com/notifications/unsubscribe-auth/AAJyv9ki3nwNIsCPoWoLN00fdGPgQfUAks5qqlcwgaJpZM4J-IsG).

@AnneGilles

This comment has been minimized.

Show comment
Hide comment
@AnneGilles

AnneGilles Sep 16, 2016

Contributor

@datakurre right! docs.plone.org is the better place for this.

Contributor

AnneGilles commented Sep 16, 2016

@datakurre right! docs.plone.org is the better place for this.

@jensens

This comment has been minimized.

Show comment
Hide comment
@jensens

jensens Nov 7, 2016

Member

may we better just rebase and merge this and then improve further?

Member

jensens commented Nov 7, 2016

may we better just rebase and merge this and then improve further?

@AnneGilles

This comment has been minimized.

Show comment
Hide comment
@AnneGilles

AnneGilles Nov 19, 2016

Contributor

@thet once you are at it, you may give the docs some love?

Contributor

AnneGilles commented Nov 19, 2016

@thet once you are at it, you may give the docs some love?

@jensens

This comment has been minimized.

Show comment
Hide comment
@jensens

jensens Dec 9, 2016

Member

I merge this now. Better now than never!

Member

jensens commented Dec 9, 2016

I merge this now. Better now than never!

@jensens jensens merged commit 52cfc6e into master Dec 9, 2016

4 checks passed

Changelog verifier Entry found
Details
Plone Contributors Agreement verifier All users have signed it
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details

@jensens jensens deleted the mephisto_253_docs branch Dec 9, 2016

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