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

feat: add logger module to the SDK #7267

Merged
merged 13 commits into from
Oct 7, 2024
Merged

Conversation

dawid-ziobro
Copy link
Contributor

This PR is adding a logger module to the SDK. Then it will be used within @vue-storefront/next and @vue-storefront/nuxt packages.

@dawid-ziobro dawid-ziobro requested a review from a team as a code owner October 4, 2024 08:26
Copy link

changeset-bot bot commented Oct 4, 2024

🦋 Changeset detected

Latest commit: 95b9c1c

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@vue-storefront/sdk Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@dawid-ziobro dawid-ziobro marked this pull request as draft October 4, 2024 08:41
@dawid-ziobro dawid-ziobro marked this pull request as ready for review October 4, 2024 09:20
Copy link
Collaborator

@bartoszherba bartoszherba left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do not forget to add changeset!

@dawid-ziobro dawid-ziobro merged commit 4e5e858 into feat/logger Oct 7, 2024
8 of 9 checks passed
bartoszherba added a commit that referenced this pull request Nov 7, 2024
* feat: logger 1.0.0-rc0

* chore: add changeset

* ci: release (rc) (#7266)

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

* feat: logger rc.1

* ci: release (rc) (#7268)

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

* feat: logger 1.0.0-rc.2

* ci: release (rc) (#7272)

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

* feat: add logger module to the SDK (#7267)

* ci: release (rc) (#7273)

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

* feat: logger rc.3

* feat: rc.3

* ci: release (rc) (#7277)

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

* feat: middleware logger (#7269)

* feat: injecting logger and metadata

* feat: injecting logger in hooks, onCreate, init fn, removed consola

* refactor: tidying, adding and adjusting tests

* refactor: applied CR tips

* chore: fix sdk test

* refactor: implemented CR suggestions

---------

Co-authored-by: Bartosz Herba <bartoszherba@gmail.com>

* ci: release (rc) (#7278)

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

* feat: add "class-methods-use-this": "off" rule (#7274)

* ci: release (#7275)

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

* CLOUD-3578 Improve Kubernetes Probes docs (#7264)

* CLOUD-3578 Improve readiness probe docs

* Update docs/content/3.middleware/2.guides/9.kubernetes-probes.md

Co-authored-by: Matt Maribojoc <mmaribojoc@vuestorefront.io>

* Add changes from Kuba

---------

Co-authored-by: Matt Maribojoc <mmaribojoc@vuestorefront.io>

* IN-3956 - Middleware extension docs is missing isNamespaced in the example code (#7271)

Co-authored-by: krystian wlodarski <kwlodarski@vuestorefront.io>

* feat: sdk logger wrapper (#7276)

* chore: update changeset

* ci: release (rc) (#7279)

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

* chore: add CR fixes

* feat: middleware logger better logger scopes (#7281)

* fix: moved global logger error to non overwritable part of code

* chore: snapshot of done work

* refactor: MethodDecoratorManager

* refactor: reduced usage of singleton to support parallel calls of fns

* fix: infinite loop constraint on types

* Revert "fix: infinite loop constraint on types"

This reverts commit 01904e3.

* chore: bumped ts jest, and jest

* fix: test

* ci: release (rc) (#7284)

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

* docs: working with logger (#7285)

* fix: merging logger configs (#7286)

* docs(tfal-56): next/nuxt logger implementation (#7282)

* docs: update next/nuxt docs (#7290)

* chore: export types from logger module (#7292)

* ci: release (rc) (#7293)

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

* feat: logger middleware error boundary (#7291)

* feat: error boundary

* refactor: refactor

* feat: use appropriate verbosity levels specifically in the browser environment (#7296)

* ci: release (rc) (#7297)

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

* chore: remove logger module (#7298)

* feat: block overwriting alokai metadata (#7300)

* feat: block overwriting alokai metadata

* refactor: outsourced deep merging of obj's to lodash.merge

* feat: guard to prevent overwriting alokai key in metadata obj

* fix: tests

* Update packages/middleware/src/logger/injectMetadata.ts

Co-authored-by: Bartosz Herba <bherba@vuestorefront.io>

---------

Co-authored-by: Bartosz Herba <bherba@vuestorefront.io>

* ci: release (rc) (#7303)

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

* feat: replace console with alokai logger in the middleware module (#7299)

* feat: change log level to verbosity (#7306)

* ci: release (rc) (#7307)

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: Bartosz Herba <bartoszherba@gmail.com>

* docs(tfal-89,tfal-83): update logger docs for current solution (#7308)

* docs(tfal-89,tfal-83): update logger docs for current solution

* docs: fix

* docs: alokai metadata (#7305)

* docs: block overriding alokai key in provided metadata

* Apply suggestions from code review

Co-authored-by: daaf <44862757+dawid-ziobro@users.noreply.github.com>

* Apply suggestions from code review

---------

Co-authored-by: daaf <44862757+dawid-ziobro@users.noreply.github.com>
Co-authored-by: Bartosz Herba <bherba@vuestorefront.io>

* chore: remove custom handler docs

* chore: exit pre mode

* feat: update logger exports and docs

* chore: update docs after cr

* docs: fix sdk docs

* feat: change log structured interface parameters

* Apply suggestions from code review - missing type annotation

Co-authored-by: daaf <44862757+dawid-ziobro@users.noreply.github.com>

* chore: update node env for logger unit tests

---------

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: daaf <44862757+dawid-ziobro@users.noreply.github.com>
Co-authored-by: Filip Jędrasik <filip.jdrasik@gmail.com>
Co-authored-by: Artur <5359825+sethidden@users.noreply.github.com>
Co-authored-by: Matt Maribojoc <mmaribojoc@vuestorefront.io>
Co-authored-by: Krystian Wlodarski <kryswlo95@gmail.com>
Co-authored-by: krystian wlodarski <kwlodarski@vuestorefront.io>
Co-authored-by: Dawid Ziobro <dziobro@vuestorefront.io>
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