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

Feature parity with snoowrap #46

Closed
thislooksfun opened this issue Mar 3, 2022 · 2 comments
Closed

Feature parity with snoowrap #46

thislooksfun opened this issue Mar 3, 2022 · 2 comments
Assignees
Labels
closed-during-archival This issue/PR was open when this repo was archived. good first issue Good for newcomers help wanted Extra attention is needed

Comments

@thislooksfun
Copy link
Owner

thislooksfun commented Mar 3, 2022

There are still several large areas of the codebase that need work before snoots has feature-parity with snoowrap. For a full class-by-class rundown of what is missing see below. If you have a specific needs that snoots doesn't currently meet, please let me know so I can prioritize accordingly!

Snoowrap Classes:

Notes/caveats:

  • This is comparing snoowrap@1.23.0 against snoots@v1.0.0-dev.22.
  • The names used here are snoowrap's names. snoots' naming scheme is often different.
  • I tried to be as complete as possible but it's likely I missed something.
  • This does not cover any method parameters, just the methods themselves.
  • This is a live list, it will be updated as compatibility improves.

snoowrap

RedditContent (base class for all items)

ReplyableContent (extends RedditContent)

✅ Full feature parity.

VoteableContent (extends ReplyableContent)

  • Missing properties
    • author_flair_background_color
    • author_flair_css_class
    • author_flair_richtext
    • author_flair_template_id
    • author_flair_text_color
    • author_flair_text
    • author_flair_type
    • author_fullname1
    • downs1
    • no_follow
    • removal_reason
    • report_reasons
    • subreddit_id1
    • subreddit_name_prefixed1
    • ups1

Comment (extends VoteableContent)

  • Missing properties
    • collapsed_reason
    • controversiality
    • depth
  • Missing methods

RedditUser (extends RedditContent)

Submission (extends VoteableContent)

  • Missing properties
    • clicked
    • content_categories
    • link_flair_background_color
    • link_flair_css_class
    • link_flair_richtext
    • link_flair_template_id
    • link_flair_text_color
    • link_flair_text
    • link_flair_type
    • media_embed
    • media_only
    • media
    • parent_whitelist_status
    • post_hint
    • preview
    • previous_visits
    • pwls2
    • quarantine
    • removal_reason
    • removed_by_category
    • secure_media_embed
    • secure_media
    • selftext_html
    • selftext
    • thumbnail
    • view_count
    • visited
    • whitelist_status
    • wls2
  • Missing Methods

LiveThread (extends RedditContent)

❌ Unimplemented

PrivateMessage (extends ReplyableContent)

❌ Unimplemented

Subreddit (extends RedditContent)

MultiReddit (extends RedditContent)

❌ Unimplemented

ModmailConversation (extends RedditContent)

❌ Unimplemented

ModmailConversationAuthor (extends RedditContent)

❌ Unimplemented

WikiPage (extends RedditContent)

❌ Unimplemented

Listing

✅ Feature parity, but implemented very differently.


1This will not be implemented.
2This is unlikely to be implemented.
3This relates to private messaging, which is an ongoing discussion.

@thislooksfun thislooksfun added help wanted Extra attention is needed good first issue Good for newcomers labels Mar 3, 2022
@thislooksfun thislooksfun self-assigned this Mar 3, 2022
@thislooksfun thislooksfun pinned this issue Mar 3, 2022
@thislooksfun thislooksfun added the closed-during-archival This issue/PR was open when this repo was archived. label Jun 13, 2023
@thislooksfun thislooksfun closed this as not planned Won't fix, can't repro, duplicate, stale Jun 13, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
closed-during-archival This issue/PR was open when this repo was archived. good first issue Good for newcomers help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

1 participant