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

Make properties "required" for Radius resources #6944

Closed
vinayada1 opened this issue Dec 11, 2023 · 3 comments
Closed

Make properties "required" for Radius resources #6944

vinayada1 opened this issue Dec 11, 2023 · 3 comments
Labels
bug Something is broken or not working as expected important This item is a high priority Issue we intend to address as soon as possible triaged This issue has been reviewed and triaged

Comments

@vinayada1
Copy link
Contributor

vinayada1 commented Dec 11, 2023

Bug information

With the conversion to typespec, by default the properties are not required on the resource. This can cause issues like: #6882

#6889 has been created to make properties as "required" on the applications resource and specifically fix issue #6882. However, we need to revisit the typespec schema for other resources and make the properties as required in all other cases where this is a problem.

AB#10740

@vinayada1 vinayada1 added the bug Something is broken or not working as expected label Dec 11, 2023
@radius-triage-bot
Copy link

👋 @vinayada1 Thanks for filing this bug report.

A project maintainer will review this report and get back to you soon. If you'd like immediate help troubleshooting, please visit our Discord server.

For more information on our triage process please visit our triage overview

@shalabhms shalabhms added the triaged This issue has been reviewed and triaged label Dec 14, 2023
@radius-triage-bot
Copy link

👍 We've reviewed this issue and have agreed to add it to our backlog. Please subscribe to this issue for notifications, we'll provide updates when we pick it up.

We also welcome community contributions! If you would like to pick this item up sooner and submit a pull request, please visit our contribution guidelines and assign this to yourself by commenting "/assign" on this issue.

For more information on our triage process please visit our triage overview

@shalabhms shalabhms added the important This item is a high priority Issue we intend to address as soon as possible label Dec 14, 2023
@radius-triage-bot
Copy link

We've prioritized work on this issue. Please subscribe to this issue for notifications, we'll provide updates as we make progress.

We also welcome community contributions! If you would like to pick this item up sooner and submit a pull request, please visit our contribution guidelines and assign this to yourself by commenting "/assign" on this issue.

For more information on our triage process please visit our triage overview

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something is broken or not working as expected important This item is a high priority Issue we intend to address as soon as possible triaged This issue has been reviewed and triaged
Projects
None yet
Development

No branches or pull requests

3 participants