Skip to content

Add #[\AllowDynamicProperties] where needed#119

Open
mtolhuys wants to merge 1 commit into
Asana:masterfrom
mtolhuys:master
Open

Add #[\AllowDynamicProperties] where needed#119
mtolhuys wants to merge 1 commit into
Asana:masterfrom
mtolhuys:master

Conversation

@mtolhuys

@mtolhuys mtolhuys commented Aug 24, 2023

Copy link
Copy Markdown

Potential solution for issue and Closes #118

@dellow

dellow commented Aug 25, 2023

Copy link
Copy Markdown

Please merge this. Causing constant warning in our log files

@MichelleBlanchette

Copy link
Copy Markdown

@mtolhuys Thanks for providing this PR because I can at least apply its patch file in my build scripts and deployment pipelines.

Patch file for this PR can be accessed via GitHub's URL which is https://patch-diff.githubusercontent.com/raw/Asana/php-asana/pull/119.patch if anyone else is interested in doing the same.

@tristanisham

Copy link
Copy Markdown

Came to make this request myself. See it's been languishing for a long time. Thanks, Asana.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

Creation of dynamic property warnings

4 participants