fetch authenticated users posts

writeas / writeas-cli

Command line client for Write.as / WriteFreely

https://write.as/apps/cli

GNU General Public License v3.0

113 stars 20 forks source link

fetch authenticated users posts #29

Closed ghost closed 5 years ago

ghost commented 5 years ago

This changes the 'posts' subcommand, previously 'list', to also list any remote posts for the user if currently authenticated.

adds DoFetchPosts to the api pkg which returns all the logged in users posts.
adds a RemotePost type to the api pkg which stores some of the return information from DoFetchPosts for use in printing to stdout
adds GetUserPosts to the api pkg which converts the incoming writeas.Posts into the smaller RemotePost type
adds getExcerpt which takes the incoming writeas.Post.Content and returns a trimmed down excerpt of the content. Max 2 lines of 80 chars and delimited by '...'
changes CmdList from pkg commands to CmdListPosts which now writes out any local and remote posts in simple formatted tables

closes #13

ghost commented 5 years ago

I think I want to change the Location header to something like Type or Visibility.

There should be something like:

unclaimed (currently ref'd as local) anonymous and not linked
claimed (currently ref'd as remote) anonymous and linked to acct, maybe without a collection
published (currently ref'd as remote)

What are your thoughts @thebaer ?

thebaer commented 5 years ago

I'm open to using different language, I just need to think about it a little more, since we'll want to use the same verbiage across all our clients. I'll have an answer for you tomorrow.

ghost commented 5 years ago

re: language, we can keep it the same for now and change it around later. I've just found so far that it has been hard to differentiate.

another quick idea for this:

Local
unsynced  f93jd9efj...

Account
unsynced     jf4jjj0...
synced         f94-odfvpdf...
synced         fj4438wys90...

thebaer commented 5 years ago

Yeah, I do like the synced / unsynced terminology, as I feel it's pretty common. It's what I went with in the web and Android apps, and I haven't heard about any confusion with it in the past 3 years.

(To see what I mean with the web app, you can log out, publish a post, then log in and publish another post. Then check your posts page -- it'll show the "unsynced" post at the bottom.)

ghost commented 5 years ago

almost done, I just want to add tests for getExcerpt

ghost commented 5 years ago

ok all set for final review, oops wrong matt. @thebaer

thebaer commented 5 years ago

Looks great. Thanks, @robjloranger!