/docs

All links should be relative #1315

Merged
merged 1 commit into from Aug 30, 2016

Conversation

Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
3 participants
@marcoceppi @bac @pmatulis
@marcoceppi
Owner

marcoceppi commented Aug 29, 2016

fixes #1313

@marcoceppi marcoceppi changed the title from All links should be relative fixes #1313 to All links should be relative Aug 29, 2016

@bac
Contributor

bac commented Aug 30, 2016

👍
@pmatulis
Contributor

pmatulis commented Aug 30, 2016

Looks ok but for the future please prepend './' to the links. This is what we have settled on throughout the docs.

@pmatulis pmatulis merged commit 9d4d95f into juju:master Aug 30, 2016

@bac
Contributor

bac commented Aug 30, 2016

@pmatulis Could someone schedule a review of all docs to make sure we don't have other absolute URLs lurking?
@pmatulis
Contributor

pmatulis commented Aug 30, 2016

In master,

$ grep -r jujucharms.com * | grep docs

clouds-LXD.md:See [General configuration options](https://jujucharms.com/docs/stable/config-general)
contributing.md:[jujucharms.com](http://jujucharms.com/docs). Its source documents are easy to
howto-drupal-iis.md:Started](https://jujucharms.com/docs/getting-started.html) section to get
howto-node.md:Started](https://jujucharms.com/docs/getting-started.html) section and
reference-release-notes.md:  and Azure. See https://jujucharms.com/docs/devel/storage for more information.
reference-release-notes.md:  documented at https://jujucharms.com/docs/devel/wip-storage.
reference-release-notes.md:  documented at https://jujucharms.com/docs/devel/wip-storage.
reference-release-notes.md:  https://jujucharms.com/docs/1.20/authors-charm-actions
reference-release-notes.md:  https://jujucharms.com/docs/stable/config-vagrant
temp-release-notes.md:https://jujucharms.com/docs/devel/controllers
temp-release-notes.md:    https://jujucharms.com/docs/devel/controllers
temp-release-notes.md:See: https://jujucharms.com/docs/devel/controllers-creating
temp-release-notes.md:See: https://jujucharms.com/docs/devel/clouds
temp-release-notes.md:See: https://jujucharms.com/docs/devel/clouds
temp-release-notes.md:See: https://jujucharms.com/docs/devel/credentials
temp-release-notes.md:https://jujucharms.com/docs/devel/controllers
temp-release-notes.md:https://jujucharms.com/docs/devel/models
temp-release-notes.md:To bootstrap models using the LXD, manual, and MAAS providers, see the special clouds section of: https://jujucharms.com/docs/devel/clouds
temp-release-notes.md:https://jujucharms.com/docs/devel/users
temp-release-notes.md:https://jujucharms.com/docs/devel/users-auth
temp-release-notes.md:https://jujucharms.com/docs/devel/users-creating
temp-release-notes.md:https://jujucharms.com/docs/devel/users-manage
temp-release-notes.md:a bundle of charms. See: https://jujucharms.com/docs/devel/charms-bundles
temp-release-notes.md:GUI charm. See: https://jujucharms.com/docs/devel/controllers-gui
temp-release-notes.md:supporting multiple series. See: https://jujucharms.com/docs/devel/charms-deploying
temp-release-notes.md:The new LXD provider is the best way to use Juju locally. See: https://jujucharms.com/docs/devel/clouds-LXD
temp-release-notes.md:[documentation](https://jujucharms.com/docs/devel/authors-charm-store#entities-explained)
@bac
Contributor

bac commented Aug 30, 2016

Thanks for taking a look @pmatulis. Should we open a new issue to take care of all of those, or just reopen this one?
@pmatulis
Contributor

pmatulis commented Aug 30, 2016 •
Edited 1 time

@bac Open another issue.

@pmatulis pmatulis referenced this pull request Aug 30, 2016

Merged

backport developer-layers.md from master to 2.0 #1321

@pmatulis
Contributor

pmatulis commented Aug 30, 2016

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