Skip to content
This repository has been archived by the owner on Jul 26, 2023. It is now read-only.

Ability to dedupe Pocket archive #34

Closed
lissertations opened this issue Jun 29, 2020 · 4 comments · Fixed by #36
Closed

Ability to dedupe Pocket archive #34

lissertations opened this issue Jun 29, 2020 · 4 comments · Fixed by #36
Assignees
Labels
enhancement New feature or request

Comments

@lissertations
Copy link

Pickpocket is great, but it only works on one's pocket list, not archive. It would be useful to be able to dedupe an archive as well (particularly when used in conjunction with pocket-snack)

@hughrun
Copy link
Owner

hughrun commented Jun 29, 2020

Good idea. This should be a simple matter of adding a flag to indicate whether to dedupe the List or the Archive, but I'll probably have to refactor the whole thing a bit to do it properly - which isn't necessarily a bad thing.

@hughrun hughrun self-assigned this Jul 5, 2020
@hughrun
Copy link
Owner

hughrun commented Jul 6, 2020

Thinking about this more @lissertations it may make more sense to move this whole script into ‘pocketsnack’ as a pair of options (‘—dedupe —archive’ or ‘—dedupe —list)

@lissertations
Copy link
Author

At the risk of creating more work for you, that might be a good way to go. Does this mean the deduping could only happen in conjunction with 'snacking'? Or could this process run independently?

@hughrun
Copy link
Owner

hughrun commented Jul 7, 2020

It would run independently but using the same program. I'll have to think about this but maybe I'll split the functions out into a proper module that could be imported independently (i.e. from another script).

@hughrun hughrun transferred this issue from hughrun/pickpocket Jul 7, 2020
@hughrun hughrun added the enhancement New feature or request label Jul 7, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants