-
Notifications
You must be signed in to change notification settings - Fork 135
feat: Mention screen locking security #1987
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
Conversation
|
Preview for this PR was built for commit |
gippy
left a comment
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 urls seem to be incorrect.
sources/platform/console/index.md
Outdated
| ## Adding different authentication methods | ||
|
|
||
| After you create your account, you might still want to use the other authentication methods. To do that, go to your [account settings](https://console.apify.com/account/settings) and scroll down to the **Login** section. There, you will see all available authentication methods and their configuration. | ||
| After you create your account, you might still want to use the other authentication methods. To do that, go to your [account settings](https://console.apify.com/settings/account) and scroll down to the **Login** section. There, you will see all available authentication methods and their configuration. |
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.
This is not correct right? The url is https://console.apify.com/settings/security and the Login section is second on the page, so you do not really need to scroll down.
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.
Good point, I've just replaced the urls that were redirected, didn't think about the context.
|
Preview for this PR was built for commit |
|
@gippy Thanks :D It should be fine now, I've rephrased it everywhere as "Go to Login & Privacy section of your account settings" |
|
Preview for this PR was built for commit |
TC-MO
left a comment
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.
Just one small change
Co-authored-by: Michał Olender <92638966+TC-MO@users.noreply.github.com>
TC-MO
left a comment
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.
LGTM
|
Preview for this PR was built for commit |
No description provided.