Skip to content

Commit

Permalink
feat: add conditional checks for document / window
Browse files Browse the repository at this point in the history
Refactor `XMLHttpRequest` in `send` to use `fetch`

update transport tests

fix test suite

fix linting issues

fix linting issues

downgrade jest version number

package downgrade

fix test suite

cleanup comment

fix package version

linting

add comments on Cloudflare workers

add comments on Cloudflare workers

fix references to document

ts-standard for easier diff

remove async option

working on diffs

working on diffs

working on diffs

working on diffs

working on diffs

remove async

one more time

tests: add worker tests

tests: add worker tests

tests: add worker tests

use jasmine

use jasmine

remove jasmine from worker

remove jasmine from worker

one last try

circumvent fetch

add some suggestions
  • Loading branch information
KonnorRogers committed Dec 6, 2022
1 parent 2a4dd4e commit 1941244
Show file tree
Hide file tree
Showing 21 changed files with 4,570 additions and 1,245 deletions.
3 changes: 3 additions & 0 deletions .gitignore
Expand Up @@ -6,3 +6,6 @@ node_modules
*/**/dist
*/**/.DS_Store
*/**/node_modules

lerna-debug.log
browserstack.err
3 changes: 1 addition & 2 deletions packages/core/src/client.ts
Expand Up @@ -12,7 +12,7 @@ import {
filterUrl,
formatCGIData,
getSourceForBacktrace,
runAfterNotifyHandlers, endpoint, isBrowserConfig, getCauses
runAfterNotifyHandlers, endpoint, getCauses
} from './util'
import {
Config,
Expand Down Expand Up @@ -206,7 +206,6 @@ export abstract class Client {
endpoint: endpoint(this.config.endpoint, '/v1/notices/js'),
maxObjectDepth: this.config.maxObjectDepth,
logger: this.logger,
async: isBrowserConfig(this.config) ? this.config.async : undefined,
}, payload)
.then(res => {
if (res.statusCode !== 201) {
Expand Down
1 change: 0 additions & 1 deletion packages/core/src/types.ts
Expand Up @@ -127,7 +127,6 @@ export type TransportOptions = {
endpoint: string,
maxObjectDepth?: number,
logger: Logger
async?: boolean // don't like this here because it's only for browser
}

export type NoticeTransportPayload = {
Expand Down

0 comments on commit 1941244

Please sign in to comment.