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

Make XataApiClient to use ES Proxies #1287

Merged
merged 19 commits into from
Jan 9, 2024
Merged

Make XataApiClient to use ES Proxies #1287

merged 19 commits into from
Jan 9, 2024

Conversation

SferaDev
Copy link
Member

No description provided.

Signed-off-by: Alexis Rico <sferadev@gmail.com>
Copy link

changeset-bot bot commented Dec 15, 2023

🦋 Changeset detected

Latest commit: f6c7a51

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

This PR includes changesets to release 10 packages
Name Type
@xata.io/client Major
@xata.io/cli Patch
@xata.io/codegen Major
@xata.io/importer Patch
@xata.io/plugin-client-cache Patch
@xata.io/plugin-client-cloudflare Patch
@xata.io/drizzle Patch
@xata.io/kysely Patch
@xata.io/netlify Patch
@xata.io/plugin-client-opentelemetry Patch

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

Signed-off-by: Alexis Rico <sferadev@gmail.com>
Copy link
Contributor

github-actions bot commented Dec 15, 2023

Your pull request has been published to npm.

You can install @xata.io/client by running:

npm install @xata.io/client@0.0.0-alpha.vd271933d3d9508845438fce1c4c4cc578ab52f96

Other packages are published like this:

npm install @xata.io/mypackage@0.0.0-alpha.vd271933d3d9508845438fce1c4c4cc578ab52f96

To test the CLI, run:

npx @xata.io/cli@0.0.0-alpha.vd271933d3d9508845438fce1c4c4cc578ab52f96

[Pull request has been updated, please wait for latest changes to be pushed to npm...]

Copy link
Contributor

github-actions bot commented Dec 15, 2023

size-limit report 📦

Path Size
packages/client/dist/index.mjs 14.09 KB (-11.43% 🔽)
packages/client/dist/index.cjs 15.17 KB (-10.84% 🔽)
packages/codegen/dist/index.mjs 1.98 MB (+0.02% 🔺)
packages/codegen/dist/index.cjs 1.99 MB (-0.06% 🔽)

Signed-off-by: Alexis Rico <sferadev@gmail.com>
This reverts commit ca1153f.

Signed-off-by: Alexis Rico <sferadev@gmail.com>
Signed-off-by: Alexis Rico <sferadev@gmail.com>
@@ -88,7 +88,7 @@ describe('API Client Integration Tests', () => {
tableName: 'table'
},
body: {
record: { email: 'example@foo.bar' }
email: 'example@foo.bar'
Copy link
Contributor

@eemmiillyy eemmiillyy Dec 18, 2023

Choose a reason for hiding this comment

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

We may have had a bug here and didn't notice. I don't think this part of the API was/should be changed. The input here is not typesafe.

Copy link
Member Author

Choose a reason for hiding this comment

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

I just added this test on Friday, weird I didn't catch it

@eemmiillyy eemmiillyy marked this pull request as ready for review December 18, 2023 12:59
@eemmiillyy eemmiillyy requested a review from a team as a code owner December 18, 2023 12:59
@SferaDev SferaDev changed the base branch from main to next December 19, 2023 07:45
@xata-bot xata-bot force-pushed the next branch 3 times, most recently from 413b115 to 30bf0ec Compare December 19, 2023 13:29
xata-bot pushed a commit that referenced this pull request Apr 22, 2024
SferaDev added a commit that referenced this pull request Apr 23, 2024
xata-bot pushed a commit that referenced this pull request Apr 25, 2024
SferaDev added a commit that referenced this pull request Apr 29, 2024
xata-bot pushed a commit that referenced this pull request Apr 29, 2024
SferaDev added a commit that referenced this pull request Apr 30, 2024
xata-bot pushed a commit that referenced this pull request Apr 30, 2024
xata-bot pushed a commit that referenced this pull request Apr 30, 2024
xata-bot pushed a commit that referenced this pull request Apr 30, 2024
xata-bot pushed a commit that referenced this pull request May 3, 2024
xata-bot pushed a commit that referenced this pull request May 3, 2024
xata-bot pushed a commit that referenced this pull request May 3, 2024
xata-bot pushed a commit that referenced this pull request May 3, 2024
SferaDev added a commit that referenced this pull request May 3, 2024
xata-bot pushed a commit that referenced this pull request May 3, 2024
SferaDev added a commit that referenced this pull request May 6, 2024
SferaDev added a commit that referenced this pull request May 17, 2024
xata-bot pushed a commit that referenced this pull request May 17, 2024
xata-bot pushed a commit that referenced this pull request May 17, 2024
SferaDev added a commit that referenced this pull request May 29, 2024
xata-bot pushed a commit that referenced this pull request May 29, 2024
@SferaDev SferaDev mentioned this pull request May 30, 2024
eemmiillyy pushed a commit that referenced this pull request Jun 3, 2024
SferaDev added a commit that referenced this pull request Jul 4, 2024
xata-bot pushed a commit that referenced this pull request Jul 5, 2024
xata-bot pushed a commit that referenced this pull request Jul 5, 2024
xata-bot pushed a commit that referenced this pull request Jul 5, 2024
SferaDev added a commit that referenced this pull request Jul 9, 2024
xata-bot pushed a commit that referenced this pull request Jul 11, 2024
SferaDev added a commit that referenced this pull request Sep 12, 2024
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