App stops working on NC26

marxlaml commented 1 year ago

Many reviews on the Apple App Store have reported the app not working on NC26, I have the very same issue. I can authorize to the NC server, but from that point onward I cannot open any note. The single exception is when the "Offline Mode" is enabled, which defeats the purpose of the app.

Is this known? And if not what can I do to investigate this further?

jasondaigo commented 1 year ago

Never tested before NC26 but I can confirm it doesn’t work right now. After login only error message

gdureuil commented 1 year ago

Not working on NC26 too. Same error as in screenshots.

marxlaml commented 1 year ago

I looked a bit further into this:

When I try to open a note on the iOS app nothing happens (not even an error message)
Instead I get an error on my Nextcloud server saying "Editor text is unknown"
This error comes from the DirectEditing Manager
The exception is thrown when an editorId is not found in the list of editors
The list of editors is empty when called

Also:

When I test this using the Android app it works flawlessly
The Android app does not even call that function, which is making me think some kind of workflow is outdated in the iOS app

P.S.: Is this app even mantained anymore? The last commit is 2 months old and the negative reviews on the AppStore are not stopping because of issues such as this one (and apparently more issues which are similar in nature)

juliushaertl commented 1 year ago

cc @marinofaggiana This seems to be an issue when the text app is not enabled or available for direct editing, which should be indicated in the capabilities.

marinofaggiana commented 1 year ago

@juliushaertl https://github.com/nextcloud/notes-ios/blob/720b1aa119f0fe315bd985c7d0970bfbbda83392/iOCNotes/NotesTableViewController.swift#L361

juliushaertl commented 1 year ago

Are capabilities refetched somehow? Could it be that the old ones are still stored?

marinofaggiana commented 1 year ago

No, every time read the capabilities ...

marinofaggiana commented 1 year ago

https://github.com/nextcloud/notes-ios/pull/42

juliushaertl commented 1 year ago

Question to the reporters, are you using server side encryption with or without a master key?

(ref https://github.com/nextcloud/text/issues/3762#issuecomment-1643647718)

marxlaml commented 1 year ago

No server side encryption here (and obviously therefore no master key)

jasondaigo commented 1 year ago

Was this supposed to be addressed in any way by todays update ? I installed the update but the error remains. im not using encryption either

nextcloud / notes-ios

App stops working on NC26 #37