Skip to content
This repository was archived by the owner on Dec 20, 2024. It is now read-only.

modify the comment of DstIP #1533

Open
timyinshi wants to merge 3 commits into
base: master
Choose a base branch
from
Open

modify the comment of DstIP #1533

timyinshi wants to merge 3 commits into from

Conversation

@timyinshi
Copy link

@timyinshi timyinshi commented Mar 10, 2021

Signed-off-by: timyinshi shiguangyin@inspur.com

modify the comment of DstIP

@timyinshi
modify the comment of DstIP
07b9b2b 
Signed-off-by: timyinshi <shiguangyin@inspur.com>
@jim3ma
Copy link
Member

jim3ma commented Mar 10, 2021

LGTM

@jim3ma
Copy link
Member

jim3ma commented Mar 10, 2021 •
edited
Loading

can you update apis/swagger.yml with your change:

      dstIP:
        type: "string"
        description: |
          the peer ID of the target Peer.

and docs/api_reference/apis.md:

|**dstIP**  <br>*optional*|the peer ID of the target Peer.|string|

@jim3ma
Copy link
Member

jim3ma commented Mar 10, 2021 •
edited
Loading

docs/api_reference/apis.md need update.

@timyinshi
modify the error comment of dstIP in apis.md
8e2391f 
Signed-off-by: timyinshi <shiguangyin@inspur.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@timyinshi @jim3ma