Feature domain tags #31
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.

Hi there @mmagnus,
I've recently changed my workflow and I don't use tags anymore. Therefore, all the articles appear under "untagged", which I didn't like.
Therefore, I've developed a new feature that organizes the articles under sections based on the first level domain of the URL of each article. Check this screenshot:
However, there are two drawbacks:
First of all, this feature is incompatible with the pocket tags normal functionality (TAGS, TAGS_EXCEPTIONS, INCLUDE_UNTAGGED, etc.). This is because the recipe ignores duplicates, and therefore an article can't belong in a section inherited from a pocket tag and also belong in a section created from its domain. Therefore, if SECTIONS_BY_DOMAIN=True pocket tags will be ignored. Because of that, I have left it disabled by default, so it won't break current use cases.
And second, because I use the tld module for extracting the domain, now the recipe has a new dependency (pip install tld). Therefore, even if someone is not going to use it python would check the import line and raise an exception. To avoid that, I have commented the related import. This can be resolved in a way better elegant way (lazy load, use a regular expressión, etc.) but I don't have the time right now.
Therefore, if someone wants to use this functionality you have to:
https://github.com/mmagnus/Pocket-Plus-Calibre-Plugin/tree/feature_domain_tags