Skip to content

Conversation

@toastts
Copy link
Contributor

@toastts toastts commented Nov 30, 2022

moved the peers-related functionality into the /peering-peers endpoint, added the admin auth to the route, updated the godocs

resolves #577

@gmelodie gmelodie added the APIv2 API v2 changes label Nov 30, 2022
@gmelodie gmelodie changed the title /peering-peers endpoint api-v2 /peering-peers endpoint Nov 30, 2022
Copy link
Contributor

@gmelodie gmelodie left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for doing this!

Co-authored-by: Gabriel Cruz <8129788+gmelodie@users.noreply.github.com>
@toastts
Copy link
Contributor Author

toastts commented Dec 20, 2022

@gmelodie forgot to ping, this PR should be ready now; lmk if there's any more changes

Copy link
Contributor

@gmelodie gmelodie left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Since this is for APIv2, could you move these to /api/v2/api.go and /api/v2/handlers.go instead?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

APIv2 API v2 changes

Projects

Status: In Progress

Development

Successfully merging this pull request may close these issues.

3 participants