Add history search commits based on user

[olemr2]

The feature request

We are 15+ developers, and being able to filter on user would have been great.

Proposed solution

Add history search commits based on user

Additional context

No response

[anaseeem]

@pol-rivero If you think this is a good direction for this fork, I’d love to take a shot at it. It would definitely add a lot of value for larger teams.

Also do you know if upstream (desktop/desktop) has already implemented or discussed this? I searched and didn't see anything. If not, let me know if I should go ahead.

[pol-rivero]

I'm not sure about the UX side of this.
@olemr2 are you proposing searching by the committer name directly? Wouldn't that return tons of results?

[olemr2]

Probably. A cap at 2-3 weeks would cover most use-cases I can think of where we have missed it.
Second thought. Merge commits are most interesting. Those would be fewer?

[pol-rivero]

Sorry, but I don't think adding arbitrary N-week caps would be a good idea.

[olemr2]

Scrolling through the History unfiltered already produces 'tons of results', so I fail to see the concern?

[anaseeem]

@pol-rivero I think he wants the history search to be like this screenshot taken from github swebsite issues tab. It doesn't have to be exactly like how it is in github website, but you will get the idea. Correct me if I am wrong @olemr2.

Well the history tab already shows all of the results, so filtering by an author will definitely get less results if not same as before. So, it could be implemented and I think upstream(desktop/desktop) is not going to have this anytime soon.

[olemr2]

Yes, exactly.
Just to make it easier to track down what an author has been up to lately.

[pol-rivero]

Okay, I see. Then, I assume syntax would be:

author:some.email@my-org.com Some text query

I'd filter by email, since the name can contain spaces.
Does that sound right?

[anaseeem]

@pol-rivero Yeah, that sounds right. Would you like me to take a shot at implementing this, or were you planning to tackle it yourself? Let me know, thank you.

[pol-rivero]

@anaseeem Sure! I'll leave this to you if you want to give it a shot :)