Skip to content

Conversation

jtroussard
Copy link
Contributor

Updating the Linkedin example so that it follows the new pattern provided by #530

@jtroussard jtroussard requested a review from JonathanHuot April 21, 2024 18:23
@jtroussard jtroussard added this to the 2.0.1 milestone Apr 21, 2024
@jtroussard
Copy link
Contributor Author

@JonathanHuot might need some guidance on the scope of the unit test, also maintainer access to the vault might be more granular, however I have not tried to push anything to secrets using the cli yet.

@coveralls
Copy link

Coverage Status

coverage: 90.097%. remained the same
when pulling cf7420d on docs-examples-linkedin
into 77905a4 on master.

@jtroussard jtroussard marked this pull request as draft April 21, 2024 18:26
@JonathanHuot
Copy link
Contributor

Wow thanks for the PR let me review.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants