Skip to content
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

chore: notice.causes refactor #854

Merged
merged 1 commit into from
Aug 11, 2022
Merged

Conversation

subzero10
Copy link
Member

Status

READY

Description

Refactor notice.causes outside the Notice type, since it's only used when sending the payload to Honeybadger.

Todos

  • Tests

@subzero10 subzero10 requested a review from shalvah August 10, 2022 16:07
@subzero10 subzero10 self-assigned this Aug 10, 2022
@subzero10
Copy link
Member Author

subzero10 commented Aug 10, 2022

Hey @shalvah, while I was merging master to the monorepo branch, I realized that I could move causes from the Notice to NoticeTransportPayload, then call getCauses in buildPayload. The tests are passing. Is this OK for you?
image

@shalvah
Copy link
Contributor

shalvah commented Aug 10, 2022

Yeah, that would work.

@subzero10 subzero10 merged commit 03752b5 into monorepo Aug 11, 2022
@shalvah shalvah deleted the monorepo_causes_refactor branch August 11, 2022 19:42
subzero10 added a commit that referenced this pull request Aug 12, 2022
)

* feat(monorepo): setup monorepo skeleton and break js package to 2 packages

* chore(monorepo): fix test scripts

* chore(monorepo): fix test scripts

* chore(monorepo): rollup config

* chore(monorepo): remove ie11 from integration testing

Refs: https://blogs.windows.com/windowsexperience/2022/06/15/internet-explorer-11-has-retired-and-is-officially-out-of-support-what-you-need-to-know/

* chore(monorepo): set org name to hb-test (temporarily)

* chore(monorepo): bump to v4.0.5 to match master

* chore(monorepo): generate changelog files

* chore(monorepo): publish without asking

* chore(monorepo): publish config => public

* chore(monorepo): revert test org name

* chore(monorepo): add readme files

* chore(monorepo): core readme file

* chore(monorepo): modify readme

* chore(monorepo): improve ts build

* chore(monorepo): fix tsd

* Update README.md

* chore: remove eslint comment

* docs: make README clearer

* chore: refactor causes out of notice class (#854)

Co-authored-by: Joshua Wood <josh@joshuawood.net>
Co-authored-by: shalvah <diakon.ng@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants