-
Notifications
You must be signed in to change notification settings - Fork 2.7k
[msal-common] Configuration and Client #1325
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
Conversation
…to msal-node-base
/lib/.DSStore should be ignored |
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.
Left a couple comments that should be addressed, otherwise looks good. +1 for tests, great job!
@DarylThayil The ignore rules for MSAL specific ignore filesnode_modules/ Doesn't this include |
A pattern for .DS_Store files should be added to the root gitignore (doesn't appear to be there), and the ds_store file in this commit should be deleted. |
Ah! I was looking at the wrong place! |
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.
Looks good now!
…ccount-authority-folders
…ders [msal-common] Create `account` and `authority` folders
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.
Update from dev to make sure all added tests are passing, I will approve once this is done.
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.
Builds are still failing but approving for now.
This PR is the first for
msal-node
and does the following:config
andapp
modulesThis will be followed up with a
PublicClientApplication
formsal-node
andPublicClientSPAApplication
formsal-browser
working. Currently this branch is compilable only withmsal-common
.