-
Notifications
You must be signed in to change notification settings - Fork 63
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: check-ins configuration api #1241
Changes from 11 commits
cbcae61
4660140
9bcf5b7
c03194e
0ae46ef
15dfa7c
a76c985
494c309
051649b
eb2d00f
4aec048
4073ae4
746efd2
45a2a8b
34e39af
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,5 +1,5 @@ | ||
import { Client as BaseClient } from '../src/client' | ||
import { Config, Logger, Notice, Noticeable, Transport, TransportOptions, NoticeTransportPayload, UserFeedbackFormOptions } from '../src/types' | ||
import { Config, Logger, Notice, Noticeable, Transport, TransportOptions, UserFeedbackFormOptions } from '../src/types' | ||
|
||
export function nullLogger(): Logger { | ||
return { | ||
|
@@ -12,7 +12,10 @@ export function nullLogger(): Logger { | |
} | ||
|
||
export class TestTransport implements Transport { | ||
send(_options: TransportOptions, _payload: NoticeTransportPayload): Promise<{ statusCode: number; body: string }> { | ||
defaultHeaders(): Record<string, string> { | ||
return {}; | ||
} | ||
send<T>(_options: TransportOptions, _payload: T): Promise<{ statusCode: number; body: string }> { | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Made the |
||
return Promise.resolve({ body: JSON.stringify({ id: 'uuid' }), statusCode: 201 }); | ||
} | ||
} | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,20 @@ | ||
module.exports = { | ||
personalAuthToken: process.env.HONEYBADGER_PERSONAL_AUTH_TOKEN, | ||
checkins: [ | ||
{ | ||
name: 'Weekly Exports', | ||
slug: 'weekly-exports-custom-slug', | ||
projectId: process.env.HONEYBADGER_PROJECT_ID, | ||
scheduleType: 'simple', | ||
reportPeriod: '1 week', | ||
gracePeriod: '10 minutes' | ||
}, | ||
{ | ||
name: 'Hourly Notifications', | ||
projectId: process.env.HONEYBADGER_PROJECT_ID, | ||
scheduleType: 'simple', | ||
reportPeriod: '1 hour', | ||
gracePeriod: '5 minutes' | ||
} | ||
] | ||
} |
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Project that demonstrates |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,4 @@ | ||
import Honeybadger from '@honeybadger-io/js' | ||
|
||
console.log('Hello from checkins-manager') | ||
console.log(Honeybadger.config) |
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,15 @@ | ||
{ | ||
"name": "checkins-manager", | ||
"version": "1.0.0", | ||
"description": "an example that demonstrates checkins configuration api", | ||
"main": "index.mjs", | ||
"scripts": { | ||
"start": "node index.mjs", | ||
"test": "echo \"Error: no test specified\" && exit 1" | ||
}, | ||
"dependencies": { | ||
"@honeybadger-io/js": "file:../.." | ||
}, | ||
"author": "", | ||
"license": "ISC" | ||
} |
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.
Support more than one client (
CheckinsClient
andHoneybadgerClient
)