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

Adds the release name (target update version) #95

Merged
merged 2 commits into from
Apr 16, 2024

Conversation

fraga
Copy link
Contributor

@fraga fraga commented Apr 15, 2024

Following new updated json file that adds the target update version to the upcoming calendar view model

@FH-Inway
Copy link
Collaborator

Awesome, thank you!

Would be nice to add it to the "Upcoming Updates" form as well. Let me know if you want to work on that and if you want to do it in this pr.

image

image

@fraga
Copy link
Contributor Author

fraga commented Apr 16, 2024

@FH-Inway sure ! I never actually navigated to that form I just always exported it. Any suggestions for the name? Release name seems just too long:
Release Name
image

Version
image

@FH-Inway
Copy link
Collaborator

Any suggestions for the name? Release name seems just too long:

I'm good with either option. Would prefer Release name, just because it maps to what is displayed in LCS and the name of the data object. I get the "too long" argument, but since we already have several other columns where the header is longer than one line, I don't think it really matters. Your choice :)

Thanks for taking on the additional work. ❤️

@fraga
Copy link
Contributor Author

fraga commented Apr 16, 2024

There you go the designer on VS 2022 changed a lot, let me know if you need me to fix anything before merging.

@FH-Inway FH-Inway merged commit e29b8c4 into microsoft:master Apr 16, 2024
2 checks passed
@FH-Inway
Copy link
Collaborator

Thanks again, new version 0.50.0 is released.

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.

2 participants