Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[VCM] introduce EOL warning infobar #21474

Merged
merged 4 commits into from
Oct 27, 2022

Conversation

yuyoyuppe
Copy link
Contributor

Summary of the Pull Request

"Learn more" is clickable and leads to the issue.

image

PR Checklist

Validation Steps Performed

@Jay-o-Way
Copy link
Collaborator

Hm, just yesterday I saw the issue and it gave me the idea that it was still debatable, but looks pretty definitive...

Copy link
Contributor

@niels9001 niels9001 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One minor comment.. LGTM otherwise!

src/settings-ui/Settings.UI/Views/VideoConference.xaml Outdated Show resolved Hide resolved
Copy link
Collaborator

@jaimecbernardo jaimecbernardo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I like that it links to a discussion issue where we specify the version we're removing it :)
LGTM! Good work!

@yuyoyuppe
Copy link
Contributor Author

Thanks for the help everyone!

@yuyoyuppe yuyoyuppe dismissed niels9001’s stale review October 27, 2022 14:03

@niels9001 already fixed all his suggestions

@yuyoyuppe yuyoyuppe merged commit c6d31cc into microsoft:main Oct 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants