Skip to content
This repository has been archived by the owner on Nov 3, 2020. It is now read-only.

Update links to docs #161

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

anne-edwards
Copy link
Contributor

Karl spotted that these were wrong - I've just updated them.

Karl spotted that these were wrong.
@@ -1,3 +1,3 @@
# The SpatialOS Online Services: Service account CLI tool

See the documentation [Online Services section of the SpatialOS documentation website](https://documentation.improbable.io/online-services/docs), specifically the [Service account CLI tool documentation](https://documentation.improbable.io/online-services/docs/platform-service-account-cli).
See the documentation [Online Services section of the SpatialOS documentation website](https://online-services.readme.io/docs/welcome), specifically the [Service account CLI tool documentation](https://online-services.readme.io/docs/platform-service-account-cli).

Choose a reason for hiding this comment

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

These links present me with a login page. That right?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Hmm, good point. I think the plan was to give access to anyone who needed it, via a ReadMe account. I'll give you access now - or at least try to (with ReadMe it's never as easy as it should be).

Choose a reason for hiding this comment

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

Wait, so we have to invite users to create a readme account to read these docs? Can these docs not be public, given that the tools it is documenting are public on Github?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

That's another good point. I'm not 100% sure of the background to this move, but from what you're saying, it sounds like we can keep these docs public but just not have them on our main docs site. Will double-check with the other TWs first thing tomorrow in case I've missed some context, but it sounds like it should be fine.

@tomtkarl
Copy link

Lots of broken docs links in this repo, fwiw: https://github.com/spatialos/online-services/search?q=online-services&type=code

@spatialos spatialos deleted a comment from anne-edwards Oct 30, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants