-
Notifications
You must be signed in to change notification settings - Fork 1
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
Docs, example and ease of use suggestion for acquireTokenSilent #2
Comments
I suppose this could be cleaner: const [response] = createResource(
tokenRequestConfig,
async (tokenRequestConfig) => await writeContext.instance.acquireTokenSilent(tokenRequestConfig)
) |
Yeah, I think writing a small arrow function would make more sense than an a very simple wrapper for a very specific scenario. It also makes it clearer what's happening with the inline arrow function (a token is being acquired based on the config). |
Based on how I have used the library would be happy to put a few docs together to show how to use this. Are you interested in a PR? |
For sure, you'd be very welcome to do that! I think the best course of action would be a readme entry, or possibility within doc comments in the codebase. |
Raised PR here: #3 |
What are your thoughts?
The text was updated successfully, but these errors were encountered: