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 device service name level to Add Event POST API #799

Closed
cloudxxx8 opened this issue Feb 16, 2023 · 1 comment · Fixed by #801
Closed

Add device service name level to Add Event POST API #799

cloudxxx8 opened this issue Feb 16, 2023 · 1 comment · Fixed by #801
Assignees
Labels
enhancement New feature or request

Comments

@cloudxxx8
Copy link
Member

🚀 Feature Request

Relevant Package [REQUIRED]

This feature request is for core-data

Description [REQUIRED]

There should be an additional device service name level in the Add Event POST API in v3

ApiEventProfileNameDeviceNameSourceNameRoute = ApiEventRoute + "/{" + ProfileName + "}" + "/{" + DeviceName + "}" + "/{" + SourceName + "}"

The edgex-go should also be modified if the constant name is changed
https://github.com/edgexfoundry/edgex-go/blob/c6ae7449aad5f05f69153f6120284e804cf836f6/internal/core/data/router.go#L31

@cloudxxx8 cloudxxx8 added the enhancement New feature or request label Feb 16, 2023
@cloudxxx8
Copy link
Member Author

client should also be modified

func (ec *eventClient) Add(ctx context.Context, req requests.AddEventRequest) (
dtoCommon.BaseWithIdResponse, errors.EdgeX) {
path := path.Join(common.ApiEventRoute, req.Event.ProfileName, req.Event.DeviceName, req.Event.SourceName)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants