-
Notifications
You must be signed in to change notification settings - Fork 642
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
Dart grpc client #3144
Dart grpc client #3144
Conversation
Codecov Report
@@ Coverage Diff @@
## desktop-gui #3144 +/- ##
============================================
Coverage 88.48% 88.48%
============================================
Files 239 239
Lines 12107 12107
============================================
Hits 10713 10713
Misses 1394 1394 |
ce421e3
to
815b774
Compare
815b774
to
ce14db3
Compare
ce14db3
to
c29f7ed
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey @andrei-toterman, LGTM. Thanks!
Same as the other associated PR's, I approve them now and merge only after rebasing.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Very clean and tidy implementation.
495fe6d
to
3129ce4
Compare
c29f7ed
to
8a7ebf5
Compare
3129ce4
to
e13e1a9
Compare
8a7ebf5
to
facb1c2
Compare
e13e1a9
to
4dcb44b
Compare
facb1c2
to
73905e9
Compare
4dcb44b
to
9772b55
Compare
73905e9
to
33d1867
Compare
9772b55
to
083e979
Compare
33d1867
to
58319e8
Compare
083e979
to
105e419
Compare
58319e8
to
713b7eb
Compare
105e419
to
04e4e06
Compare
713b7eb
to
6468fe8
Compare
04e4e06
to
bae61eb
Compare
6468fe8
to
93a12fe
Compare
93a12fe
to
861888c
Compare
This PR adds a wrapper for the grpc client and several useful data providers, as well as displaying some rudimentary instance information on the main page, just to make sure it works.