Bookmarks

[BlueGreenMagick]

Add bookmark_users to Work.

Work.bookmark_users returns a list of (list of usernames) and (list of pseudonyms) in case people needed to know both. ex: [ [user1, user2, ,user3], [pseuds1, pseuds2, pseuds3] ]

On additional note, it also saves session in Work.__init__ so it can be reused when getting bookmarked users.

[BlueGreenMagick]

When testing on a fiction with 2000+ bookmarks, came across an issue of server responding with Retry Later message instead. Should I add a time delay between requests? Or I could add the delay on Retry Later response.

[alexwlchan]

Thanks for the patch! I haven't used this code in a while so my knowledge is a bit foggy.

I’ve put a couple of comments inline. I’m happy to pick up the patch and finish it off, or if you’d rather go back and forth until it’s in a mergeable state, that works too.

[alexwlchan]

Isn't it provided above?

[BlueGreenMagick]

The first line of comment says that complete representation of work is provided when calling json(). But because of bookmark_users not being provided when calling json(), it is no longer a complete representation, which was the reason I wrote that line.

Perhaps it should be reworded differently?