dvanoni / notero

A Zotero plugin for syncing items and notes into Notion
https://download.notero.vanoni.dev
MIT License
2.39k stars 107 forks source link

Auto delete page when item is merged or deleted #46

Open HernandoR opened 2 years ago

HernandoR commented 2 years ago

At present, Notero could not detect no-longer-existing items, pages related to which will be left in the database.

I understand that is it much better to let people decide how to deal with their page when the item is merged or deleted. But is it possible to insert certain identifiers such as certain tags or properties so that one can create a view to check if any pages need to deal.

dvanoni commented 2 years ago

Hi @HernandoR, thanks for the feedback! It should be possible for Notero to detect when items are deleted or merged, so I can think of a couple ways to approach this:

What do you think?

theotheo commented 2 years ago

I think the second way is prefer. Because I would be upset if I lost my notes after some unintended action. I done merging a dozen times but I have no idea which Zotero item is deleted after it. So I imagine a case that I merged a couple Zotero items, but which one would be deleted and which one would be remained?

dvanoni commented 2 years ago

@theotheo, that's a great point. If we set up the syncing into Notion, in theory there would be one page that ends up with all the merged data, but you wouldn't know ahead of time which page it was going to be. I imagine this could cause issues if you had other things set up on the deleted page(s) that you wanted to keep, such as custom properties or relations with other pages.

It would probably still be a little clunky, but I agree that adding a Deleted property would likely be the easiest way to allow you to resolve any merging issues in Notion. In theory you could do something similar by looking for deleted items in the Notion trash, but I find it difficult to find specific pages in the trash. I think it would be much easier to have a database view for deleted pages.

EFFORTSUN commented 2 years ago

I wonder when this function will be available?

dvanoni commented 2 years ago

@EFFORTSUN me too! 😅 Unfortunately, I don't have a lot of time available to work on it right now, but I'll be sure to post an update here whenever this functionality is ready!

shiyang1997 commented 2 years ago

this function is very useful, hope quickly come out, thanks a lot

jerryliu20d commented 1 year ago

Before the feature is added, an alternative way is to add a tag like "trash" to it (can use Zotero-tag). Then remove it from zotero after syncing to notion. In notion you can hide all entries with "trash" tags. Or deleting all entries with "trash" tag every month. I am also looking forward to this new feature! BTW how to vote for this feature in the Project todo?

dvanoni commented 1 year ago

That's a great workaround, @jerryliu20d! Thanks for sharing! 😄

I've been manually tallying votes by keeping track of how many people either leave a comment on an issue or give it a 👍 (which I see you've done, thanks!)

alvorithm commented 1 year ago

I suggest to add a tag zotero-trash and a update the table templates on the webpage with a matching view 'Trashed in Zotero'.

TobiasDraven commented 1 year ago

Hi, I am rather concerned about the safety of my notes in Notion. Like some previous posters have mentioned, it would be catastrophic if I accidently delete or move documents in Zotero, and they get deleted/lost/destroyed/banished to the void in Notion. I agree too that rather than deleting the page in Notion, rather add a "deleted" property or something similar. Also, is it wiser to use a different database for personal study notes, and then relate that to the Notero database?

dvanoni commented 1 year ago

Thanks for the input, @TobiasDraven. I definitely agree that a "deleted" property seems like a much better approach than moving pages into the Notion trash.

As far as where to keep your personal notes, I've been making it a priority to not have Notero overwrite/delete anything (other than the properties it syncs), so I believe it should be safe to work directly in the Notero-synced database if you like. For example, the experimental note syncing that was recently released does write to the database pages in Notion, but it makes sure to do in it a separate section so that it doesn't overwrite anything else on the page.