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

array field undefined value length check #184

Merged

Conversation

sagarchoudhary96
Copy link
Contributor

No description provided.

@vercel
Copy link

vercel bot commented Oct 21, 2023

@sagarchoudhary96 is attempting to deploy a commit to the Measured Team on Vercel.

A member of the Team first needs to authorize it.

@sagarchoudhary96
Copy link
Contributor Author

#179

@sagarchoudhary96
Copy link
Contributor Author

@chrisvxd , sorry but there was one check which got missed in the previous fix. This should resolve the issue completely.

@sagarchoudhary96 sagarchoudhary96 force-pushed the fix/array-field-undefined-value branch from dde2311 to 28f1ef6 Compare October 21, 2023 06:35
@sagarchoudhary96 sagarchoudhary96 changed the title Fix/array field undefined value array field undefined value length check Oct 21, 2023
@vercel
Copy link

vercel bot commented Oct 23, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
puck-demo ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 23, 2023 1:05pm

Copy link
Member

@chrisvxd chrisvxd left a comment

Choose a reason for hiding this comment

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

Good catch, thanks again @sagarchoudhary96

@chrisvxd chrisvxd merged commit 68afd09 into measuredco:main Oct 23, 2023
2 checks passed
@sagarchoudhary96 sagarchoudhary96 deleted the fix/array-field-undefined-value branch October 23, 2023 13:11
@sagarchoudhary96
Copy link
Contributor Author

@chrisvxd what is the canary version for this change.

@chrisvxd
Copy link
Member

0.10.1-canary.68afd09

They're automatically published on each merge with the commit, in this case 68afd09. You should always be able to see it pop up a few minutes after merge on NPM: https://www.npmjs.com/package/@measured/puck?activeTab=versions

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