Skip to content

Fix issue of update more than one custom property#446

Open
roboticsexpert wants to merge 1 commit intoebess:masterfrom
roboticsexpert:fix-bug-of-more-than-one-custom-field
Open

Fix issue of update more than one custom property#446
roboticsexpert wants to merge 1 commit intoebess:masterfrom
roboticsexpert:fix-bug-of-more-than-one-custom-field

Conversation

@roboticsexpert
Copy link
Copy Markdown

as you see request attribute is overriding in first loop iteration and in second iteration it has wrong value that cause to just update first custom property field

@cv-chameleon
Copy link
Copy Markdown

@ebess can you merge this pull request please?

@roboticsexpert
Copy link
Copy Markdown
Author

roboticsexpert commented Jan 2, 2025

@cv-chameleon I really don't remember, how these lines of changes, fix the problem now, but it has worked for me and fixed the problem . you can use this instead if you are in hurry

composer require roboticsexpert/advanced-nova-media-library

@cv-chameleon
Copy link
Copy Markdown

Thank you @roboticsexpert , in the meantime I already switched to this fork https://github.com/marshmallow-packages/advanced-nova-media-library

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