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

MediaBlockService required if formats defined #2435

Open
mesiarm opened this issue Dec 20, 2023 · 5 comments
Open

MediaBlockService required if formats defined #2435

mesiarm opened this issue Dec 20, 2023 · 5 comments

Comments

@mesiarm
Copy link

mesiarm commented Dec 20, 2023

Make format key always optional even if formats are defined.
Now it is: 'required' => \count($formatChoices) > 0,
It is unable to set some of images reference if formats are defined

@VincentLanglet
Copy link
Member

HI @mesiarm, can you explain why the format key should be optional ?

@mesiarm
Copy link
Author

mesiarm commented Apr 27, 2024

HI @mesiarm, can you explain why the format key should be optional ?

There may be cases, when I want to choose it by programming logic or use full size.

@VincentLanglet
Copy link
Member

There may be cases, when I want to choose it by programming logic or use full size.

But I assume the data need to be required for people which doesn't have extra-programming-logic. Changing the field to optional will create an issue for them.

Does adding an option to configureSettings would help you ?

@mesiarm
Copy link
Author

mesiarm commented May 2, 2024

There may be cases, when I want to choose it by programming logic or use full size.

But I assume the data need to be required for people which doesn't have extra-programming-logic. Changing the field to optional will create an issue for them.

Does adding an option to configureSettings would help you ?

Yes, it would help

@VincentLanglet
Copy link
Member

PR welcomed then.

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

No branches or pull requests

2 participants