feat: display author of changes in workflow tab

[pr0grammm]

feat: display author of changes in workflow tab

Summary
Addresses #5466 : Display author of changes in workflow tab.
Have made code changes to display the post author name below the collection name inside the change card.

Test plan
This PR changes the UI by adding the author name as seen in the below screenshot:

At this time, changes have been made for GitHub backend. Need to propagate the changes for other backends and test for those as well.

Checklist

Please add a x inside each checkbox:

• I have read the contribution guidelines.
• Code is formatted via running yarn format.
• Tests are passing via running yarn test.
• The status checks are successful (continuous integration). Those can be seen below.

Cute baby elephant

[erezrokah]

Thanks for the follow up @pr0grammm!

This seems to break GitHub when using GraphQL. You can add use the following configuration to reproduce:

backend:
  name: github
  branch: <branch>
  repo: <repo>
  use_graphql: true

Also it's missing the Azure DevOps backend.

Finally, should we show the user full name instead of the login handle?

[pr0grammm]

Thanks for the follow up @pr0grammm!

👍

This seems to break GitHub when using GraphQL. You can add use the following configuration to reproduce:

backend:
  name: github
  branch: <branch>
  repo: <repo>
  use_graphql: true

Thanks, I have addressed this, please have a check. GraphQL is supported currently only for GitHub right?

Also it's missing the Azure DevOps backend

I am quite unfamiliar with this backend. I may be slow to address this.

Finally, should we show the user full name instead of the login handle?

👍. Since configuring the full name is optional there are chances for it to be empty. We can try to display the full name and if it is empty then we can fallback to the login handle maybe? Also for GitHub I am unable to find the full name returned in the API response. It seems to me only the login is present?

[erezrokah]

GraphQL is supported currently only for GitHub right?

Yes, it is

I am quite unfamiliar with this backend. I may be slow to address this.

Let me see if I can recreate my setup for Azure.

Since configuring the full name is optional there are chances for it to be empty. We can try to display the full name and if it is empty then we can fallback to the login handle maybe?

That makes sense

Also for GitHub I am unable to find the full name returned in the API response. It seems to me only the login is present?

I think so, we might be able to update the GraphQL query to get it, but for the REST API we may need to send an additional request (which is unfortunate)

[pr0grammm]

@erezrokah wonder why the CI fails?

[erezrokah]

@erezrokah wonder why the CI fails?

We use pre-recorded data to test the different backends. The GraphQL query changes means that we'll need to update the data similar to fb51377.

Let me try a find/replace to fix this

[pr0grammm]

Sure thanks a lot

[pr0grammm]

Hi @erezrokah! For GitLab and BitBucket, it seems that the full name is mandatory after all. So I have skipped checking for the login handle altogether. Hope that's okay. So now only GitHub is left to be handled...

[erezrokah]

So now only GitHub is left to be handled

Done in 39118d2.

I hope parallelizing the requests the requests will lower the performance impact of sending an additional one

[erezrokah]

🚀

[pr0grammm]

Done in 39118d2.

I hope parallelizing the requests the requests will lower the performance impact of sending an additional one

Awesome! Sorry I was not able to get to this. I think there is also a change needed in the graphql query in GitHub to retrieve the name and not just the login? I'll try to look into that this week.

[erezrokah]

I think there is also a change needed in the graphql query in GitHub to retrieve the name and not just the login?

Good point! Done in 275ee3b