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

Add option to delete notebook if linked file gets deleted #918

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

funsafe-math
Copy link

A fix for #287

@yamanq
Copy link

yamanq commented Feb 27, 2022

Tested on Android 11, works. Thanks!

@brendonscript
Copy link

Will this ever be merged @nevenz ? This is a big issue keeping a lot of people from using Orgzly

@@ -590,4 +590,6 @@
<string name="developer_options">Možnosti pro vývojáře</string>
<string name="git_repository_type">Typ Git repozitáře</string>
<string name="in_development">Ve vývoji</string>
<string name="pref_title_sync_deletion">Sync deletion</string>
<string name="pref_title_summary_sync_deletion">Delete notebook if its deleted from repo</string>
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
<string name="pref_title_summary_sync_deletion">Delete notebook if its deleted from repo</string>
<string name="pref_title_summary_sync_deletion">Delete notebook if it is deleted from repo</string>

or it's, but it is good style to write it out in textual form

same for all the others

@xeruf
Copy link

xeruf commented Jan 3, 2023

Unfortunately some restructuting has happened, raising big conflicts around here :/

@Itrekr
Copy link

Itrekr commented Mar 30, 2023

Is there any indication if this is going to get implemented within the foreseeable future? It's really annoying that it's impossible to delete notebooks remotely because orgzly will just re-upload them automatically when it can't find them on the remote server.

@mxxivcz
Copy link

mxxivcz commented Mar 30, 2023

it's impossible to delete notebooks remotely

It is now possible to delete a notebook by switching all but one of the devices to plane mode and then deleting the notebook on each of the devices.

@Itrekr
Copy link

Itrekr commented Mar 30, 2023

it's impossible to delete notebooks remotely

It is now possible to delete a notebook by switching all but one of the devices to plane mode and then deleting the notebook on each of the devices.

True @ajtakrajta, but that is rather cumbersome if you have to delete a large amount of files. For example, in my workflow I create daily notes which should be disposable. I have a function in Emacs that lets me go through those files and delete them if I no longer need them, and keep them if I do. This can easily be up to 10 files if I do this once every two weeks or so. That means I'll have to carefully select each and every one of those files on my mobile device just so they don't get re-uploaded automatically when I've deleted them on my pc.

The logical solution for dealing with such conflicts would just be to somehow log the list of note files that orgzly finds in the directory, if, during a sync it can't find one of the notes it previously had, either remove the local note or mark them as a conflict and let the user resolve them.

@ugurbolat
Copy link

Just a ping.

@danielkrajnik
Copy link

danielkrajnik commented Jun 19, 2023

+1 I hate just repeating what others said, but it's impossible to use Orgzly when it keeps recreating deleted files.

@MyFedora
Copy link

MyFedora commented Aug 13, 2023

Unfortunately some restructuting has happened, raising big conflicts around here :/

No not really.

  • Resolve conflicts for translations
  • Port changes to SyncWorker

I built an apk based on the most recent commit and the changes already, but I'm new to Orgzly, so I didn't verify whether it works yet.

Edit: Yep, works. When I delete org files from a device, Orgzly also deletes the notebook. Here's the patch: Add-option-to-delete-notebook-if-linked-file-gets-deleted.patch.txt. Remove .txt from the end of the file name.

@colonelpanic8
Copy link
Contributor

@tadeuszk733 Would you be interested in trying to merge this into the community fork at

https://github.com/orgzly-community/orgzly-android

I might consider doing it myself if you think its in a good enough state that it should be merged. WDYT?

@colonelpanic8
Copy link
Contributor

@tadeuszk733 as far as I can tell, this doesn't handle the case of two way sync. Is that correct?

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.

10 participants