Skip to content

[WIP] Major Version 5 #1861

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

Closed
wants to merge 9 commits into from
Closed

[WIP] Major Version 5 #1861

wants to merge 9 commits into from

Conversation

HazAT
Copy link
Member

@HazAT HazAT commented Jan 31, 2019

@HazAT HazAT added this to the 5.0.0 milestone Jan 31, 2019
@HazAT HazAT requested a review from kamilogorek as a code owner January 31, 2019 11:46
@getsentry-bot
Copy link
Contributor

getsentry-bot commented Jan 31, 2019

Messages
📖 ✅ TSLint passed
📖 @sentry/browser gzip'ed minified size: 21.1523 kB

Generated by 🚫 dangerJS against 160933e

@HazAT HazAT force-pushed the major/5 branch 2 times, most recently from d47622c to 4f4d5eb Compare February 15, 2019 11:29
@HazAT HazAT closed this Feb 18, 2019
@HazAT HazAT reopened this Feb 18, 2019
@HazAT
Copy link
Member Author

HazAT commented Feb 18, 2019

Draft pull request doesn't work yet 😞

kamilogorek and others added 9 commits February 21, 2019 09:45
…1858)

* ref: Move PromiseBuffer and Error to utils package

* ref: Remove all async api

* ref: Remove invokeClientAsync function

* feat: Finish Node SDK

* feat: SyncPromise implementation

* fix: Browser SDK

* fix: beforeSend callback

* fix: Core tests

* fix: linting and tests

* fix: browser npm package + add error for manual tests

* meta: Remove deprecations

* fix: Typedocs and public exposed functions

* meta: Changelog

* Apply suggestions from code review

Co-Authored-By: HazAT <[email protected]>

* fix: CodeReview

* feat: #1871
* fix: Events created from exception shouldnt have top-level message attribute (#1831)

* Use SyncPromise internally & Remove deprecated API & Trim Public API (#1858)

* ref: Move PromiseBuffer and Error to utils package

* ref: Remove all async api

* ref: Remove invokeClientAsync function

* feat: Finish Node SDK

* feat: SyncPromise implementation

* fix: Browser SDK

* fix: beforeSend callback

* fix: Core tests

* fix: linting and tests

* fix: browser npm package + add error for manual tests

* meta: Remove deprecations

* fix: Typedocs and public exposed functions

* meta: Changelog

* Apply suggestions from code review

Co-Authored-By: HazAT <[email protected]>

* fix: CodeReview

* feat: #1871

* ref: Move public interfaces to types package

Move addBreadcrumb from the client to the hub

* fix: Tests

* fix: Small type issues

* ref: Remove scope listeners

* fix: Add flush to interface

* ref: Remove interfaces from core
* ref: Move ExtendedError to a types package

* ref: Remove assign util in favor of Object.assign

* ref: Pass Event to sendEvent instead of just a body

* ref: Remove isArray and isNaN utils

* ref: Rewrite normalization and removed unused utils

* Utils rewamp changelog
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants