-
Notifications
You must be signed in to change notification settings - Fork 673
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
MG-2053 - Structure API Endpoints Based on Entities #2386
base: main
Are you sure you want to change the base?
Conversation
e676ad5
to
945ffdd
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@felixgateru , I think list things is not in good shape, Could you for Domain member can able to list the things which they created ?
2aaa9cc
to
10193b9
Compare
things/service.go
Outdated
} | ||
pm.EntityType = auth.UserType | ||
pm.EntityID = res.GetId() | ||
return svc.listEntityThings(ctx, res.GetId(), res.GetDomainId(), auth.MembershipPermission, auth.DomainType, res.GetDomainId(), pm) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If domain is disabled , then this should not work for domain non administrator users. Please check this case
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have checked this condition. When the domain is disabled only admins are able to list things
8a7c8c3
to
f3fb6d0
Compare
Signed-off-by: WashingtonKK <washingtonkigan@gmail.com> Signed-off-by: Arvindh <arvindh91@gmail.com> Co-authored-by: Arvindh <arvindh91@gmail.com> Co-authored-by: Arvindh <30824765+arvindh123@users.noreply.github.com> Signed-off-by: 1998-felix <felix.gateru@gmail.com>
Signed-off-by: 1998-felix <felix.gateru@gmail.com>
Signed-off-by: 1998-felix <felix.gateru@gmail.com>
Signed-off-by: Arvindh <arvindh91@gmail.com>
Signed-off-by: Arvindh <arvindh91@gmail.com>
Signed-off-by: 1998-felix <felix.gateru@gmail.com>
Signed-off-by: 1998-felix <felix.gateru@gmail.com>
Signed-off-by: 1998-felix <felix.gateru@gmail.com>
Signed-off-by: 1998-felix <felix.gateru@gmail.com>
f3fb6d0
to
8728d92
Compare
What type of PR is this?
This is a refactor as it restructures endpoints based on entities to improve clarity and user experience.
What does this do?
This pr removes the following endpoints:
and replaces the functionlaity with list by entity option in list
domains
,users
,groups
,channels
andthings
as documented in the api.Which issue(s) does this PR fix/relate to?
Have you included tests for your changes?
Yes, relevant tests have been added and updated.
Did you document any new/modified feature?
Yes, the api documentation has been updated.
Notes