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: if an attribute is not required, allow it to not be set #361

Merged

Conversation

JohnVillalovos
Copy link
Contributor

When using the API would get "Undefinedarray key" errors on missing custom attributes, even though those attributes were not required.

Now allow the custom attribute to be missing if not a required custom attribute.

@JohnVillalovos JohnVillalovos force-pushed the jlvillal/notset_okay branch from 17cbba4 to f8da7f5 Compare May 31, 2024 06:20
@JohnVillalovos JohnVillalovos changed the title fix: if an attribute is not required, allow it not be set fix: if an attribute is not required, allow it to not be set May 31, 2024
@JohnVillalovos JohnVillalovos force-pushed the jlvillal/notset_okay branch from f8da7f5 to 657ddb7 Compare May 31, 2024 06:25
When using the API would get "Undefinedarray key" errors on missing
custom attributes, even though those attributes were not required.

Now allow the custom attribute to be missing if not a required custom
attribute.
@JohnVillalovos JohnVillalovos force-pushed the jlvillal/notset_okay branch from 657ddb7 to a08db74 Compare May 31, 2024 06:32
@effgarces effgarces merged commit c0c4324 into LibreBooking:develop Jun 1, 2024
1 check passed
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