Skip to content
This repository has been archived by the owner on Feb 27, 2023. It is now read-only.

modify the comment of DstIP #1533

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

modify the comment of DstIP #1533

wants to merge 3 commits into from

Conversation

timyinshi
Copy link

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