michael-lazar / rtv

Browse Reddit from your terminal
MIT License
4.65k stars 274 forks source link

More of a user's pages are visible #602

Closed woorst closed 6 years ago

woorst commented 6 years ago

600

Available pages:

/u/spez/{overview,submitted,comments} /u/me/{upvoted,downvoted,hidden,saved}

/u/spez defaults to overview page

/u/saved has been removed in favor of the above.

Updated tests coming eventually :)

woorst commented 6 years ago

@michael-lazar This is ready for your review

michael-lazar commented 6 years ago

Changes look great! I tested as many edge cases as I could think of and everything worked.

I made one small change to hardcode the page titles because it was weird seeing My Saved without a noun at the end.

            title_lookup = {
                'overview': 'Overview',
                'submitted': 'Submissions',
                'comments': 'Comments',
                'saved': 'Saved Content',
                'hidden': 'Hidden Content',
                'upvoted': 'Upvoted Content',
                'downvoted': 'Downvoted Content'
            }