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

Sharepoint branch updated to work on M1/M2 macs #490

Open
wants to merge 3 commits into
base: sharepoint
Choose a base branch
from

Conversation

hakt0-r
Copy link

@hakt0-r hakt0-r commented Jun 24, 2023

Hi @snobu,

I am fairly new to using gitHub and I might have created a mess here by creating an issue, forking, commenting and pulling and I really cant figure out git!!! So please forgive me!

I spent a bit of time figuring this out and fixing a few things and got this working and I wanted to help others out so here you are with my notes and changes. I hope that this links to this issue #489 .

when running npm run build you'll get a quite a few errors that talk about missing type definition files... They are not not missing... just adding one line 
"noImplicitAny": false
in tsconfig.json file suppresses those messages.
src/LoginModules.ts:14:20 - error TS2503: Cannot find namespace 'puppeteer'.
src/TokenCache.ts:60:20 - error TS2503: Cannot find namespace 'puppeteer'.
and...
src/TokenCache.ts:99:24 - error TS2339: Property 'waitFor' does not exist on type 'Page'.

These are fixed by simple oneliner changes.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant