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

Add V2 Typings to JS SDK #108

Merged
merged 2 commits into from
Nov 13, 2024
Merged

Conversation

marcinpece-apptension
Copy link
Contributor

@marcinpece-apptension marcinpece-apptension commented Oct 31, 2024

@marcinpece-apptension marcinpece-apptension force-pushed the 2549-add-v2-typings-to-js-sdk branch 2 times, most recently from d54e068 to 893b8af Compare October 31, 2024 15:00
@marcinpece-apptension marcinpece-apptension force-pushed the 2549-add-v2-typings-to-js-sdk branch from 893b8af to 1e48a7a Compare October 31, 2024 15:49
@ViolanteCodes
Copy link
Contributor

@courcelan per Marcin, this is complementary to the PR the agency opened

Copy link
Contributor

@courcelan courcelan left a comment

Choose a reason for hiding this comment

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

@ViolanteCodes these look good, we just need to ensure that the status type doesn't get overwritten by this one... so lets merge #106 into here and then use this one to merge into master.

@ViolanteCodes ViolanteCodes force-pushed the 2549-add-v2-typings-to-js-sdk branch from a1f0c17 to 41e2fe8 Compare November 13, 2024 15:55
@ViolanteCodes ViolanteCodes merged commit 28b996c into master Nov 13, 2024
@ViolanteCodes ViolanteCodes deleted the 2549-add-v2-typings-to-js-sdk branch November 13, 2024 15:56
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.

3 participants