Skip to content

Conversation

@WojtekTheWebDev
Copy link
Collaborator

@WojtekTheWebDev WojtekTheWebDev commented Feb 21, 2024

🔗 Linked issue

IN-3860

❓ Type of change

  • 📖 Documentation (updates to the documentation, readme or JSDoc annotations)
  • 🐞 Bug fix (a non-breaking change that fixes an issue)
  • 👌 Enhancement (improving an existing functionality like performance)
  • ✨ New feature (a non-breaking change that adds functionality)
  • 🧹 Chore (updates to the build process or auxiliary tools and libraries)
  • ⚠️ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description

Added a new SDK module - moduleFromEndpoints, which is a recommended way to communicate with a ServerMiddleware.

📝 Checklist

  • I have linked an issue or discussion.
  • I have updated the documentation accordingly.

@WojtekTheWebDev WojtekTheWebDev requested a review from a team as a code owner February 21, 2024 11:30
@changeset-bot
Copy link

changeset-bot bot commented Feb 21, 2024

🦋 Changeset detected

Latest commit: fe8068d

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

@WojtekTheWebDev WojtekTheWebDev marked this pull request as draft February 21, 2024 11:30
@WojtekTheWebDev WojtekTheWebDev changed the title Feat/module from endpoints feat(IN-3860): new SDK module - moduleFromEndpoints Feb 21, 2024
@WojtekTheWebDev WojtekTheWebDev marked this pull request as ready for review February 22, 2024 13:03
@WojtekTheWebDev WojtekTheWebDev mentioned this pull request Feb 26, 2024
8 tasks
…ints.spec.ts

Co-authored-by: Tomasz Góral <tomasz.gl69@gmail.com>
WojtekTheWebDev and others added 8 commits March 4, 2024 12:27
* feat: default behaviour of HTTP Client

* remove isomorphic-fetch from depts

* full implementation of HTTP client

* test: update tests based on the review

* fix issue with URL class and urls as paths

* add test for axios

* add computed headers to http client

* update tsdocs
* feat: default behaviour of HTTP Client

* remove isomorphic-fetch from depts

* full implementation of HTTP client

* feat: add customizable error handler

* await error handler

* update error handler due to feedback

* update tsdocs

* refactor default http client

* use unknown instead of any

* add method name to error handler

* update tsdocs

* apply review suggestions
* feat: default behaviour of HTTP Client

* remove isomorphic-fetch from depts

* full implementation of HTTP client

* feat: add customizable error handler

* await error handler

* refactor: improve tsdocs

* test non-object params

* add comment

* add module types to api extractor
* feat: default behaviour of HTTP Client

* remove isomorphic-fetch from depts

* full implementation of HTTP client

* feat: add customizable error handler

* await error handler

* refactor: improve tsdocs

* fix: issue with custom methods in new module

* test: additonal tests

* fix issue with losing tsdocs

* rename file

* missing import

* include credentials

* Update packages/sdk/src/modules/moduleFromEndpoints/types.ts

Co-authored-by: Łukasz Śliwa <39009379+lsliwaradioluz@users.noreply.github.com>

---------

Co-authored-by: Łukasz Śliwa <39009379+lsliwaradioluz@users.noreply.github.com>
@WojtekTheWebDev WojtekTheWebDev changed the title feat(IN-3860): new SDK module - moduleFromEndpoints feat(IN-3860): new SDK module - middlewareModule Mar 11, 2024
@WojtekTheWebDev WojtekTheWebDev merged commit 2e83504 into main Mar 11, 2024
@WojtekTheWebDev WojtekTheWebDev deleted the feat/moduleFromEndpoints branch March 11, 2024 15:38
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.

4 participants