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.

Sign up for GitHub

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

Jump to bottom

Fix: Predefined Properties doesn't respect ctype #11141

Merged
merged 4 commits into from Jan 17, 2022
Merged

Fix: Predefined Properties doesn't respect ctype #11141

merged 4 commits into from Jan 17, 2022

Conversation

blankse
Copy link
Contributor

@blankse blankse commented Jan 7, 2022 •
edited

Steps to reproduce

  1. Add an predefined property with ctype Asset
  2. Go to an document
  3. Go to settings tab
  4. Property is available

Edit: Fix also other listings

  • Search in Settings - Predefined Properties
  • Search in Settings - Predefined Asset Metadata
  • Image Config - downloadable property
  • Predefined Document Type - ComboBox doesn't respect type

@blankse blankse mentioned this pull request Jan 7, 2022
Merged
@blankse blankse force-pushed the fix_predefined_properties branch 2 times, most recently from a26ae74 to 263f17f Compare January 7, 2022 12:36
@brusch brusch added the Bug label Jan 14, 2022
@brusch brusch added this to the 10.2.9 milestone Jan 14, 2022
@weisswurstkanone weisswurstkanone merged commit d8377fc into pimcore:10.2 Jan 17, 2022
@weisswurstkanone
Copy link
Contributor

merci!

@blankse blankse deleted the fix_predefined_properties branch January 17, 2022 13:18
@blankse
Copy link
Contributor Author

blankse commented Jan 17, 2022

@weisswurstkanone Can you merge 10.2 branch in 10.x, so I can update #11142 ?

@weisswurstkanone
Copy link
Contributor

done

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

@weisswurstkanone weisswurstkanone

Labels
Bug
Projects
None yet
Milestone
10.2.9
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@blankse @weisswurstkanone @brusch