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 issue where for field of primitive list, field name and property name should be same, else crash occurs #126

Merged
merged 1 commit into from
Feb 15, 2022

Conversation

SajuThomasPattarmatam
Copy link
Contributor

Fix issue where field name is used for fetching property type instead of property name, resulting in crash when field name is "FieldName" and property name is "fieldname", since it wont get the property type as data

… of property name, resulting in crash when field name is "FieldName" and property name is "fieldname", since it wont get the property type as data
@SajuThomasPattarmatam
Copy link
Contributor Author

@mariuskatcontentful Please let me know how to complete the checks in the ci

@mariuskatcontentful
Copy link
Collaborator

CI is currently broken if you run the tests locally on your machine that will suffice as well as i will check them before the merge as well

@makinwab makinwab merged commit e594c26 into contentful:master Feb 15, 2022
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.

None yet

3 participants