-
Notifications
You must be signed in to change notification settings - Fork 15
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
Manage Project and Manage Credentials #391
Conversation
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.
hey @ritazagoni I made some edits to align more with Nick's guidance - see slack thread.
Can you please fill in the correct info for Share Credentials
(a section I added to the manage-credentials page). See my notes in the page
|
||
![New Credential Ready](/img/lightning_new_cred_ready.png) | ||
|
||
### Share Credentials |
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.
hey @ritazagoni I made some edits to align more with Nick's guidance - see slack thread.
Can you please fill in the correct info for Share Credentials
(a section I added to the manage-credentials page). See my notes in the page
I added a |
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.
@ritazagoni links referencing other pages need fixing
|
||
![User Credentials List](/img/lightning_edit_user_credential.png) | ||
|
||
For guidance on how to set up a new Credential, head over to our [Manage Credentials](http://localhost:3000/documentation/next/manage-projects/manage-credentials) page. |
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.
@ritazagoni this linking is not right - and shouldn't include the full URL. The link should use the path in directory starting from /documentation/next/...
. So I'm pretty sure this will work: /documentation/next/manage-projects/manage-credentials
. Implement it and then test it out or lmk if you want me to test
@@ -56,4 +56,4 @@ ask on [Community](https://community.openfn.org). | |||
|
|||
### Share Credentials | |||
|
|||
@Rita to complete & confirm:: I don't _think_ this nor transferring ownership is possible yet, so we might need to advise users to share outside of OpenFn (via secure means like a password manager or encrypted messaging platform) | |||
If you own a Credential, you can choose which Project has access to it. You can update project access on your `User Profile` - just follow the steps on the [User Credentials](https://docs.openfn.org/documentation/next/manage-users/user-credentials) docs page. |
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.
@ritazagoni same comment re: wrong link. Needs to look like ``/documentation/next/manage-users/user-credentials`
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.
noted, thank you!
fix broken link
fix broken link
update share creds language
I haven't been able to run docs locally yet (on Windows), so it's not tested.