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

Unable to add folders via Files screen #1287

Closed
thimper opened this issue Dec 14, 2019 · 9 comments · Fixed by #1472
Closed

Unable to add folders via Files screen #1287

thimper opened this issue Dec 14, 2019 · 9 comments · Fixed by #1472
Labels
area/linux Linux area/macos MacOS area/webui Issues specific to interaction with ipfs-webui area/windows Windows kind/bug A bug in existing code (including security flaws) need/analysis Needs further analysis before proceeding need/maintainers-input Needs input from the current maintainer(s) P1 High: Likely tackled by core team if no one steps up

Comments

@thimper
Copy link

thimper commented Dec 14, 2019

macos : 10.13.6

Adding folders has no effect。

Using the command line ipfs add-r folders is fine

@autonome
Copy link
Contributor

Thanks for reporting!

Confirmed on Mac OS X 10.14.6 with IPFS Desktop 0.10.2

STR:

  • Open web UI to files page
  • Click [+Add]
  • Select "Folder"
  • Choose a folder

Expected: Folder is added and is visible in UI

Actual: Nothing happens

@hacdias or @lidel any ideas what caused regression?

@hacdias
Copy link
Member

hacdias commented Dec 16, 2019

Unfortunately, I have no idea and there are no errors reported in the console. In the meanwhile, you can just drag'n'drop directories to the menubar icon! That should work

@lidel
Copy link
Member

lidel commented Jan 7, 2020

Just for the record, this does not not seem to be limited to Electron.
IIUC the same problem was confirmed on Google Chrome Canary.
Tracked in ipfs/ipfs-webui#1367 (comment)

@lidel
Copy link
Member

lidel commented Jan 30, 2020

Ok I think there are two bugs at play here. Dumping my notes for future reference.

I tested on Linux with Electron/Firefox/Chromium, and:

  • if webui is opened in Electron (ipfs-desktop)

    • 💢 adding folder does not work via button (no error in Console)
    • 💢 adding folder does not work via drag&drop (error)
    • 💢 adding single file does not work via drag&drop on Linux (same startsWith error)
  • if webui is opened in Firefox 72

    • 👼 adding folder WORKS via button
    • 💢 adding folder does not work via drag&drop (error)
  • if webui is opened in Brave (Chromium 77)

    • 👼 adding folder WORKS via button
    • 💢 adding folder does not work via drag&drop (error)

Update: confirmed to be a problem on windows as well: #1346

@lidel lidel mentioned this issue Feb 3, 2020
19 tasks
@lidel lidel added help wanted Seeking public contribution on this issue area/linux Linux area/macos MacOS labels Feb 4, 2020
@lidel lidel mentioned this issue Mar 11, 2020
22 tasks
@lidel lidel added kind/bug A bug in existing code (including security flaws) P0 Critical: Tackled by core team ASAP P1 High: Likely tackled by core team if no one steps up and removed P0 Critical: Tackled by core team ASAP labels Mar 11, 2020
@lidel lidel changed the title mac can not add folders Unable to add folders via Files screen Apr 3, 2020
@lidel lidel added the area/windows Windows label Apr 3, 2020
@lidel lidel pinned this issue Apr 3, 2020
@lidel lidel added the area/webui Issues specific to interaction with ipfs-webui label Apr 3, 2020
@lidel lidel mentioned this issue Apr 27, 2020
22 tasks
@lidel
Copy link
Member

lidel commented Apr 27, 2020

I believe ipfs/ipfs-webui#1468 fixes all/most of drag&drop issues described in #1287 (comment), thank you @rafaelramalho19 :-)

When we release new webui and switch ipfs-desktop to the new version we can check if this issue can be closed (or comment what remains to be fixed).

@lidel lidel added need/analysis Needs further analysis before proceeding need/maintainers-input Needs input from the current maintainer(s) and removed help wanted Seeking public contribution on this issue labels Apr 27, 2020
@lidel
Copy link
Member

lidel commented Apr 28, 2020

@rafaelramalho19 FYSA I retested with webui v2.8.0 (PR #1457) on Linux with Electron/Firefox/Chromium, and:

  • if webui is opened in Electron (ipfs-desktop)
    • 💢 adding folder does not work via button (no error in Console)
    • 👼 adding folder WORKS via drag&drop
    • 👼 adding single file WORKS via drag&drop
  • if webui is opened in Firefox
    • 👼 adding folder WORKS via button
    • 👼 adding folder WORKS via drag&drop
  • if webui is opened in Brave (Chromium)
    • 👼 adding folder WORKS via button
    • 👼 adding folder WORKS via drag&drop

So after we merge #1457 the only thing that needs to be fixed is adding folder via button in Electron.

@rafaelramalho19
Copy link
Contributor

Thanks for testing that out @lidel.

I'll see what's wrong with the button

@hacdias
Copy link
Member

hacdias commented Apr 29, 2020

💢 adding folder does not work via button (no error in Console)

@lidel it does now thanks to @rafaelramalho19's work 😃 Thank youu

@lidel
Copy link
Member

lidel commented Apr 30, 2020

All fixed in v0.11.2
Upgrade ASAP :-)

@lidel lidel unpinned this issue Oct 7, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/linux Linux area/macos MacOS area/webui Issues specific to interaction with ipfs-webui area/windows Windows kind/bug A bug in existing code (including security flaws) need/analysis Needs further analysis before proceeding need/maintainers-input Needs input from the current maintainer(s) P1 High: Likely tackled by core team if no one steps up
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants