Fields for Provision Watcher Tab don't match the Provision Watcher DTO about edgex-ui-go HOT 4 OPEN

assign this issue to @jiekechoo

from edgex-ui-go.

I have fixed this issue yiqisoft/edgex-ui-go/ , but I can't submit a new pr. Anyone can help me?

from edgex-ui-go.

@jiekechoo you should create a new branch called issue-642 to fix this issue, and open a PR against it.
https://wiki.edgexfoundry.org/display/FA/Committing+Code+Guidelines#CommittingCodeGuidelines-BranchingConventions
you can't create a new PR because there is an existing PR against your main branch #654

from edgex-ui-go.

@jiekechoo , I have merged you PRs for the Provision Watcher fix for this issue, but not seeing what is expect from the latest built image.

Here is how I tested:

From compose builder

• make pull (to make sure all the images I have are the latest
• make run ds-onvif-camera ds-usb-camera ds-llrp ui-dev
• make get-token

From UI

• Enter token when prompted
• Navigate to Provision Watcher tab (this is what I see)
  
• Edit the first one and this is what I see:
  

Here is the Provision Watcher that should be displayed.
https://github.com/edgexfoundry/device-usb-camera/blob/main/cmd/res/provisionwatchers/generic.provision.watcher.yaml

I feel like you changes are not being reflected in the latest build, but even in the images on your PR #657, the field names are incorrect.

IdenifiersAddress => Identifiers
IdentifiersPort should be removed.

Note that:

• Identifiers is a collection of key/value pairs (ie. map[string]string)
• BlockingIdentifiers is a collection of key/value pairs where the values are a list of strings (i.e. map[string][]string)

Also not that if I select the 2nd empty item in the initial list I get this which has the Device Service and Device profiles properly listed and selected.

from edgex-ui-go.