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

feat(cli): split login and up methods in the service CLI API #239

Merged
merged 3 commits into from
Mar 8, 2022

Conversation

gigovich
Copy link
Contributor

No description provided.

// Up starts engine work in the daemon.
rpc Up(UpRequest) returns (UpResponse) {}

// Down dengine work in the daemon.
Copy link
Contributor

Choose a reason for hiding this comment

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

minor typo/comment issue

client/server/daemon.go Outdated Show resolved Hide resolved
client/server/daemon.go Outdated Show resolved Hide resolved
@gigovich gigovich force-pushed the feature-exctract-cli-from-client branch 4 times, most recently from bf4401b to bb67850 Compare February 28, 2022 18:50
@gigovich gigovich force-pushed the feature-exctract-cli-from-client branch 5 times, most recently from 57c23d4 to 3ffaf98 Compare March 5, 2022 12:06
braginini
braginini previously approved these changes Mar 5, 2022
@braginini braginini self-requested a review March 5, 2022 12:13
@gigovich gigovich force-pushed the feature-exctract-cli-from-client branch 2 times, most recently from 8ba3b5d to 8e0f655 Compare March 6, 2022 16:12
@gigovich gigovich force-pushed the feature-exctract-cli-from-client branch from 8e0f655 to d62a828 Compare March 7, 2022 15:43
@braginini braginini merged commit ef47385 into netbirdio:main Mar 8, 2022
@gigovich gigovich deleted the feature-exctract-cli-from-client branch June 2, 2023 06:53
pulsastrix pushed a commit to pulsastrix/netbird that referenced this pull request Dec 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants