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

docs: change npm org scope from @oryd to @ory #730

Merged
merged 1 commit into from
Sep 28, 2021
Merged

docs: change npm org scope from @oryd to @ory #730

merged 1 commit into from
Sep 28, 2021

Conversation

swantzter
Copy link
Contributor

https://www.npmjs.com/package/@oryd/keto-client has a notice that @oryd has changed to @ory, and https://www.npmjs.com/package/@oryd/keto-grpc-client simply doesn't exist (whereas the @ory version does)

@CLAassistant
Copy link

CLAassistant commented Sep 23, 2021

CLA assistant check
All committers have signed the CLA.

@swantzter swantzter changed the title change npm org scope from @oryd to @ory docs: change npm org scope from @oryd to @ory Sep 23, 2021
@aeneasr
Copy link
Member

aeneasr commented Sep 23, 2021

Thank you very much for your contribution to the documentation! Due to the way Docusaurus versioning works, you need to make the changes in two locations:

  • docs/versioned_docs/version-v{version}/path/to/file
  • docs/docs/path/to/file

Without making them in both locations, the changes will most likely be lost when releasing another version. Thank you! :)

@swantzter
Copy link
Contributor Author

@aeneasr it looks like the change has already been made in docs/docs and are only outdated in the versioned docs, would you like me to update in v0.5 docs too? Though that only points to https://www.npmjs.com/package/@oryd/keto-client which does exist but is deprecated, yet it's still the correct version for v0.5 from what I can see

Copy link
Member

@zepatrik zepatrik left a comment

Choose a reason for hiding this comment

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

Makes sense in this case. The new package does not have the old versions, so this is good 👍

@zepatrik zepatrik merged commit 945005f into ory:master Sep 28, 2021
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