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.

Sign up for GitHub

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

Jump to bottom

impl constructors should require *Manager to enable .get on associated resources #876

Closed
martinsawicki opened this issue Jun 22, 2016 · 1 comment
Closed

impl constructors should require *Manager to enable .get on associated resources #876

martinsawicki opened this issue Jun 22, 2016 · 1 comment
Assignees
@martinsawicki
Milestone
1.0.0-beta2

Comments

@martinsawicki
Copy link

martinsawicki commented Jun 22, 2016 •
edited
Loading

to allow for direct references to related resources
this means refactoring GroupableResourcesImpl and GroupableResourceImpl to track the manager consistently for everybody
@martinsawicki martinsawicki added this to the 1.0.0-beta2 milestone Jun 22, 2016
@martinsawicki martinsawicki self-assigned this Jun 22, 2016
@martinsawicki martinsawicki changed the title impl constructors should require *ManagementClientImpl, not the *sInner impl constructors should require *Manager to enable .get on associated resources Jun 22, 2016
@martinsawicki
Copy link
Author

done in #872

@github-actions github-actions bot locked and limited conversation to collaborators Apr 13, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@martinsawicki martinsawicki

Labels
None yet
Projects
None yet
Milestone
1.0.0-beta2
Development

No branches or pull requests
1 participant
@martinsawicki