search

pnp / pnpjs

Fluent JavaScript API for SharePoint and Microsoft Graph REST APIs
https://pnp.github.io/pnpjs/
Other
750 stars 306 forks source link

Re: Use @azure/msal-browser instead of MSAL #2832

Closed datyin closed 8 months ago

datyin commented 10 months ago

Category

Related Issues

Since MSAL is deprecated now, we should use @azure/msal-browser

What's in this Pull Request?

Refactored MSAL behavior

datyin commented 10 months ago

I've resubmitted it for Version 4 :)

bcameron1231 commented 10 months ago

I'll review this. Thanks!

bcameron1231 commented 9 months ago

Updated some missing docs, fixed eslint error. Tests appear to run locally just fine.

@patrick-rodgers / @juliemturner Any additional testing you'd like to do before merging?

juliemturner commented 8 months ago

Fixed merge conflicts and pulling with PR #2900