-
Notifications
You must be signed in to change notification settings - Fork 67
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
Refactor Client Code, part 6/7: notifications #201
Refactor Client Code, part 6/7: notifications #201
Conversation
Signed-off-by: Brandon Forster <me@brandonforster.com>
Signed-off-by: Brandon Forster <me@brandonforster.com>
Signed-off-by: Brandon Forster <me@brandonforster.com>
clients/general/client.go
Outdated
} else { | ||
gc.url = params.Url | ||
func (gc *generalRestClient) FetchConfiguration(ctx context.Context) (string, error) { | ||
url, err := gc.urlClient.Prefix() |
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.
Variable naming in prior PRs is urlPrefix
; rename url
here for consistency. Applies to any/all calls to Prefix().
Signed-off-by: Brandon Forster <me@brandonforster.com>
Signed-off-by: Brandon Forster <me@brandonforster.com>
Signed-off-by: Brandon Forster <me@brandonforster.com>
Signed-off-by: Brandon Forster <me@brandonforster.com>
Signed-off-by: Brandon Forster <me@brandonforster.com>
Signed-off-by: Brandon Forster <me@brandonforster.com>
Signed-off-by: Brandon Forster <me@brandonforster.com>
Signed-off-by: Brandon Forster <me@brandonforster.com>
35cccae
to
69f0ac3
Compare
Codecov Report
@@ Coverage Diff @@
## master #201 +/- ##
==========================================
- Coverage 48.39% 45.85% -2.55%
==========================================
Files 64 64
Lines 2093 2133 +40
==========================================
- Hits 1013 978 -35
- Misses 1002 1066 +64
- Partials 78 89 +11
Continue to review full report at Codecov.
|
PR content included in #202. |
This PR is the sixth part of several working on #196.
This PR refactors all of notification's client methods.