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

Cannot import from folder #105

Open
tr1as3c opened this issue Oct 30, 2024 · 10 comments
Open

Cannot import from folder #105

tr1as3c opened this issue Oct 30, 2024 · 10 comments

Comments

@tr1as3c
Copy link

tr1as3c commented Oct 30, 2024

I have a problem importing url from folder. Whenever I click on the Import button to get the folder url, I get a Puppeteer Error.

image

Sharedown version:
Current release 5.3.4

URL Structure (doesn't require sign-in)
https://xxx-my.sharepoint.com/:f:/p/mohamed_anwar/Eq43lzuJHE5IrHnC7_Ou1KUBvg2NJ-pp6sMJvt03xSocfg

Thanks in advance!

@kylon
Copy link
Owner

kylon commented Oct 30, 2024

can you try this?
sharedownt.zip

  • install latest nodejs
  • unpack
  • open a terminal in unpacked folder
  • run npm i or yarn install
  • run npm start or yarn start

@tr1as3c
Copy link
Author

tr1as3c commented Oct 30, 2024

thank you for your prompt response.

the folder now opens, but sharedown keeps loading:
image

after a while I get another puppeteer error:
image

the same scenario happens when I select a video:
image

@tr1as3c
Copy link
Author

tr1as3c commented Oct 30, 2024

I'm not sure if this worth mentioning, but when I first ran npm i then started sharedown it gave me this error:
image

so, I had to manually copy the puppeteer folder from the installed version to the node_modules folder of the version we are testing now.

@kylon
Copy link
Owner

kylon commented Oct 30, 2024

ah missing hidden file in the zip, https://github.com/kylon/Sharedown/blob/main/.puppeteerrc.cjs

anything in logs?

@tr1as3c
Copy link
Author

tr1as3c commented Oct 30, 2024

sharedownLog.log
runPuppeteerGetURLListFromFolder: error

@kylon
Copy link
Owner

kylon commented Oct 30, 2024

to download a single video, dont use import but add url and use url from context menu->copy

this adds more logs
sharedownt2.zip

@tr1as3c
Copy link
Author

tr1as3c commented Oct 30, 2024

after multiple tries, the only solution that worked is to download videos one by one.
it didn't work directly because it requires to sign in to download the video.
the workaround that worker with me to bypass that is to:

  1. keep the sign in tab opened.
  2. open the folder url in a new tab.
  3. copy the desired video url from context menu.
  4. paste video url in the first tab that requires sign in.
    fortunately, after seconds sharedown recognizes the video and download starts.

thanks so much for helping me solving this issue.

@kylon
Copy link
Owner

kylon commented Oct 30, 2024

if you are going that way, you can just add all urls at once in add url input and enable settings to keep chrome data and brower open to avoid 2fa issues, this way it will be automatic

@tr1as3c
Copy link
Author

tr1as3c commented Oct 30, 2024

thanks for the suggestion. downloading bulk videos now will be easier and more practical.
is there an option to select and add all videos inside the folder to download queue in one step?

@kylon
Copy link
Owner

kylon commented Oct 30, 2024

its the import from folder

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

No branches or pull requests

2 participants