obsidian-copilot icon indicating copy to clipboard operation
obsidian-copilot copied to clipboard

Mobile Issue: Cannot Save Chat

Open apislogic opened this issue 1 year ago • 6 comments

  • [x ] Disable all other plugins besides Copilot (required)
  • [x ] Screenshot of note + Copilot chat pane + dev console added (required) image

Copilot version: 2.7.4

Describe how to reproduce In Obsidian Mobile (v 1.7.7) chats do not save if the designated "Default Conversation Folder Name" already exists. The save fails with the error:

plugin:copilot:159469 Error saving chat as note: Error: Folder already exists. at t. (app.js:1:750805) at app.js:1:237226 at Object.next (app.js:1:237331) at a (app.js:1:236049)

If the folder designated as ""Default Conversation Folder Name" does not already exist, the save works as expected. However, subsequent updates/new chat saves then do now work after.

Expected behavior

Expected behavior is for copilot chats to save/update on demand or on new chat creation.

Screenshots See this Loom for a demo: https://www.loom.com/share/ea356400ce8441a782b964c9acd6cd32?sid=4fbbf8f0-17b1-46ed-bd51-68b034b3ca11

apislogic avatar Dec 01 '24 19:12 apislogic

Probably because the util from Obsidian API that checks for folder existence did not work on mobile (android in this case?)

logancyang avatar Dec 01 '24 22:12 logancyang

I am getting the error on the desktop.

Screenshot 2024-12-02 at 4 59 26 AM

jrossi avatar Dec 02 '24 09:12 jrossi

I am getting the error on the desktop.

Screenshot 2024-12-02 at 4 59 26 AM

What OS? What's your folder structure?

logancyang avatar Dec 02 '24 10:12 logancyang

Mac OSX 15.1 (24B83) Obsidian: Version 1.7.7 (Installer 1.5.3)

Screenshot 2024-12-02 at 5 13 01 AM

I changed it to a top level folder as I thought my Johnney.decimal folder setup my be an issue. Before trying this out. it was set to: 00-09 System/01 Inbox/01.03 Copilot chat logs

jrossi avatar Dec 02 '24 10:12 jrossi

Yes. On Android.

On Sun, Dec 1, 2024 at 2:15 PM Logan Yang @.***> wrote:

Probably because the util from Obsidian API that checks for folder existence did not work on mobile (android in this case?)

— Reply to this email directly, view it on GitHub https://github.com/logancyang/obsidian-copilot/issues/884#issuecomment-2510274742, or unsubscribe https://github.com/notifications/unsubscribe-auth/AA4REI5B4BFD4LBICIB2OKD2DODBTAVCNFSM6AAAAABSZ72P2WVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKMJQGI3TINZUGI . You are receiving this because you authored the thread.Message ID: @.***>

apislogic avatar Dec 02 '24 18:12 apislogic

This is the same as the bug here, I believe: https://github.com/logancyang/obsidian-copilot/issues/628

In that bug @logancyang stated this was an Obsidian API bug, and that they could reproduce it on Windows but not their Mac. So, by my count, this bug affects Android, Windows, and Linux.

upnix avatar Dec 05 '24 01:12 upnix

If this is still an issue please reopen

logancyang avatar Aug 11 '25 18:08 logancyang