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

fix: typescript definition for optional data attributes #51

Merged
merged 2 commits into from
Apr 20, 2023

Conversation

AlexisAnzieu
Copy link
Contributor

@AlexisAnzieu AlexisAnzieu commented Nov 22, 2022

typing is failing because of bad function parameters

@AlexisAnzieu AlexisAnzieu changed the title make fields as optionnal as documention define make fields as optionnal as documention defines Nov 22, 2022
@levibostian levibostian changed the title make fields as optionnal as documention defines fix: make fields as optional as documention defines Dec 1, 2022
@levibostian levibostian requested review from ami-aman and xtreem88 and removed request for ami-aman December 1, 2022 16:44
@ami-aman
Copy link
Collaborator

ami-aman commented Dec 6, 2022

@AlexisAnzieu Thanks for opening the PR and contributing to customerio-reactnative package. I would like to understand better by what is meant by

typing is failing

@levibostian
Copy link
Contributor

Thank you very much for contributing to the project, @AlexisAnzieu! This change is fantastic.

Wanted to give you an update that we are performing internal testing of this change before we merge it in and ship it.

Again, thanks for making this project better.

@levibostian levibostian changed the base branch from develop to main February 23, 2023 15:08
@levibostian levibostian changed the title fix: make fields as optional as documention defines fix: typescript definition for optional data attributes Apr 20, 2023
@levibostian levibostian merged commit 4cec62a into customerio:main Apr 20, 2023
github-actions bot pushed a commit that referenced this pull request Apr 20, 2023
### [2.3.1](2.3.0...2.3.1) (2023-04-20)

### Bug Fixes

* typescript definition for optional data attributes ([#51](#51)) ([4cec62a](4cec62a))
@levibostian
Copy link
Contributor

Thank you again, @AlexisAnzieu! Greatly appreciate your PR ❤️

@AlexisAnzieu AlexisAnzieu deleted the fix_optionnal_type branch June 1, 2023 14:57
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