perfect. Here it is mine, waiting for yours
https://github.com/discourse/discourse/pull/4726
4 Likes
sorry I didn’t update one of the above posts with valid order values, can you double check and make sure your pr is using the new order values:
created
last_emailed
seen
username
email
trust_level
days_visited
posts_read
topics_viewed
posts
read_time
sure but i will anyway double check and test it when your PR will be merged
1 Like
@designbygio Okay my pr has been merged into master.
2 Likes
many thanks @blake!
Sorting is working perfectly now and my PR is ready to be merged
5 Likes
While we’re on the topic… I wonder if we could have a ‘last post date’ added as well? Would be good way to quickly gauge activity over time.
Ok, closing this now that it is done! thanks @designbygio !
4 Likes