search

integrations / microsoft-teams

Bringing your code and work to the conversations you care about with the GitHub and Microsoft integration
443 stars 97 forks source link

Option to switch channel integration to event stream rather than cards #201

Open michaeljones opened 2 years ago

michaeljones commented 2 years ago

Is your feature request related to a problem? Please describe.

(I'm using the "frustrated" wording here as it is suggested in the template.)

I'm frustrated when a pull request is rendered as a large card which goes off the top of the screen as its appearance and any updates that move it to the bottom push all other content out of the view. I miss updates if two or three PRs with large cards change status or have comments because I'm not inclined to scroll up past these large cards.

The cards are also unwanted noise. The desire to see that things have changed. I do not wish to see all the details of the PR description as you can read those by clicking through to the PR which you're almost certainly going to do anyway if you're interested.

Describe the solution you'd like Previous hand written integrations that I've done just have a one or two line message per event that happens on any of the PRs. No juggling large cards or posts in order of recent updates, just a new message per event. One or two lines per message. What was done. Who did it. The title of the PR if it is new or if it is approved or removed or something. The content of the comment (and I guess the PR title) if a comment is added though truncated down to an acceptable size if it is too large.

Unfortunately, I can't write a custom integration in my current role as the infrastructure is locked down.

Describe alternatives you've considered None others have crossed my mind.

Additional context I have looked for options in the settings. I don't think this is currently possible. It feels like the current implementation maximises on showing off what is possible when integrating with Teams but ends up with a less than desirable experience for some people. I imagine managers that don't want to load GitHub might like the current set up but it is less appealing to me when I'm on GitHub a lot already and happy to read the details there. I just want a notification stream, I guess.

ashokirla commented 2 years ago

@michaeljones Thanks for the feedback. We have plans to improve the PR card with expand and collapse functionality so that it occupies less space and yet provide all the info you need. Here is an experimentation mock we are working on. Let me know if you have any suggestions.

Screenshot 2022-04-25 at 1 26 40 PM
michaeljones commented 2 years ago

Thanks for the reply @ashokirla. That would definitely be an improvement. It looks good to me. I only really care about the title and maybe the approval status so that should work.

I still think that a event stream mode would be valid but I realise that you're not necessarily looking change it all up too much.