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

Page metadata use is not appropriate #2478

Open
SammyOina opened this issue Oct 19, 2024 · 0 comments
Open

Page metadata use is not appropriate #2478

SammyOina opened this issue Oct 19, 2024 · 0 comments

Comments

@SammyOina
Copy link
Contributor

https://github.com/absmach/magistrala/blob/eb881690c74b6f2357115848f5ba152533561d30/pkg/sdk/go/sdk.go#L90-L125

Page metadata is used by many methods on SDK interface, however, this can be confusing as struct members are selectively used by different methods. Struct members passed to a method should be options that will be used in that method.

A good example of how this should be done is:
https://github.com/moby/moby/blob/b0632b23457112c87b28be1850a17bfb7f9a9eac/client/image_create.go#L17
https://github.com/moby/moby/blob/b0632b23457112c87b28be1850a17bfb7f9a9eac/client/image_list.go#L19

Different methods have different options that apply to each method.

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

No branches or pull requests

1 participant