hodgesmr / mastodon_digest

A Python script that aggregates recent popular posts from your Mastodon timeline
BSD 3-Clause "New" or "Revised" License
439 stars 58 forks source link

Specifying username is potentially unnecessary #21

Closed adamghill closed 1 year ago

adamghill commented 1 year ago

I think mastodon_username at https://github.com/hodgesmr/mastodon_digest/blob/3f2ea4d28961592edb3650f0b9125209cad5129a/run.py#L48 is unnecessary because you can get the logged in account via mastodon.me() (as specified in https://mastodonpy.readthedocs.io/en/stable/15_everything.html#mastodon.Mastodon.me). I removed the username requirement for https://fediview.com by doing this.

There might be a use case where someone wants to log in with one account, but filter out interactions from another account, but that seems like an edge case?

Let me know if you'd like a PR for this and I'd be happy to create one.

hodgesmr commented 1 year ago

Great find! I'll get this patched shortly. Thank you!