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

Add listSubscriptions to resourceApi typings #1703

Merged
merged 2 commits into from
Mar 7, 2024
Merged

Add listSubscriptions to resourceApi typings #1703

merged 2 commits into from
Mar 7, 2024

Conversation

nturinski
Copy link
Member

Implementation needs to be done in Resource Groups, which will be coming soon

@nturinski nturinski merged commit ba5691a into main Mar 7, 2024
4 checks passed
@nturinski nturinski deleted the utils/api branch March 7, 2024 23:12
@bwateratmsft
Copy link
Contributor

Shouldn't the auth package be used instead?

@nturinski
Copy link
Member Author

Maybe... Here's my scenario:

I need a way to list the filtered subscriptions to create my own SubscriptionListStep for a user to select their sub. This is happening in the Functions and Storage extension. Since I don't think we'd want the client extensions to directly consume the auth package, I was planning on having Resources handle listing the subs out to the clients.

Resources also controls the tenant/subscription filter settings, so I thought it made more sense for it to deliver the list to the clients.

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.

3 participants