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

Implement Response.Throw() #4

Draft
wants to merge 15 commits into
base: users/annelo/synapse-rbac-llc
Choose a base branch
from

Conversation

annelo-msft
Copy link
Owner

@annelo-msft annelo-msft commented Aug 23, 2021

Investigation for Azure#23372

{
/// <summary>
/// </summary>
public class AzureError
Copy link
Owner Author

Choose a reason for hiding this comment

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

Per arch board, we won't formalize this yet

annelo-msft pushed a commit that referenced this pull request Dec 15, 2023
* Update autorest.md file with latest swagger and allow model duplication

* Generate and upgrade sdk code

* Customize redis common configuration code to include aad-enabled

* Regenerate API file

* Customize redis common configuration code to include aad-enabled

* Mitigate breaking change

* Create empty test-resources.json

* Create FlushFunctionalTests.cs

* Update assets.json

* Add functional tests for aad (#2)

Adds functional tests for aad

* Update Channel Tests (#3)

* Oscarba/fix (#4)

* Update Channel Tests

* Update correct data type

* Push regenerated code

* UpdateChannelFix (#5)

* Update Channel Tests

* Update correct data type

* Update test

* Update tag for assets after push for recordings

* Address reviewer comments

* Update generatred code after mergin in main

* Update FTs for updated models

* Update AAD FT

* Update assets.json

* Update changelog

---------

Co-authored-by: Kanchan Joshi <kanchanjoshi@microsoft.com>
Co-authored-by: Austin Tolani <austintolani@microsoft.com>
Co-authored-by: Oscar Barbosa Aquino <32723476+Darfeikus@users.noreply.github.com>
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.

1 participant