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

Manage Project and Manage Credentials #391

Merged
merged 15 commits into from
Feb 12, 2024
Merged

Manage Project and Manage Credentials #391

merged 15 commits into from
Feb 12, 2024

Conversation

ritazagoni
Copy link
Member

I haven't been able to run docs locally yet (on Windows), so it's not tested.

Copy link
Member

@aleksa-krolls aleksa-krolls left a 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
Copy link
Member

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

@ritazagoni
Copy link
Member Author

I added a user credentials page and made updates across the two.

Copy link
Member

@aleksa-krolls aleksa-krolls left a 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.
Copy link
Member

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.
Copy link
Member

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`

Copy link
Member Author

Choose a reason for hiding this comment

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

noted, thank you!

@aleksa-krolls aleksa-krolls merged commit 21819eb into main Feb 12, 2024
1 check failed
@aleksa-krolls aleksa-krolls deleted the project branch February 12, 2024 06:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants