-
Notifications
You must be signed in to change notification settings - Fork 114
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
[v3] updating docs with new license management flow and UX #864
Conversation
🎊 PR Preview has been successfully built and deployed to https://localstack-docs-preview-pr-864.surge.sh 🎊 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Few more comments:
- The terminology is a bit unclear. Is it
Auth Token
orauth token
? I guess we should go with the former consistently. - Rather than deleting the pages (which might trouble other users), let's put up a deprecation notice for the time being.
- The docs are unclear about the path for API keys. Have they been deprecated? What's the timeline wherein they still work? And more questions.
@@ -0,0 +1,73 @@ | |||
--- | |||
title: "Managing Users and Licenses" | |||
tags: ["LocalStack Pro", "Accounts"] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We can remove the tags. Please add a linkTitle
here
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
what should the linkTitle then be?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The same as title
af3fb03
to
a0b18f0
Compare
Merge when v3 is released!
don't merge before rolling out changes in front-end https://github.com/localstack/localstack-web/pull/1001
Todo