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

Define allowed settings in ingredients #2425

Merged
merged 1 commit into from
Feb 1, 2023

Conversation

tvdeyen
Copy link
Member

@tvdeyen tvdeyen commented Feb 1, 2023

What is this pull request for?

Ingredients have lots of settings they allow to be defined in the definition.

In order to have an overwiew we document them on the class.

Checklist

  • I have followed Pull Request guidelines
  • I have added a detailed description into each commit message
  • I have added tests to cover this change

@tvdeyen tvdeyen added this to the 7.0 milestone Feb 1, 2023
@tvdeyen tvdeyen requested a review from mamhoff February 1, 2023 10:36
Ingredients have lots of settings they allow to be
defined in the definition.

In order to have an overwiew we document them on the class.
@tvdeyen tvdeyen force-pushed the document-ingredient-settings branch from 78625fb to d0753de Compare February 1, 2023 12:34
@tvdeyen tvdeyen enabled auto-merge February 1, 2023 12:34
@tvdeyen tvdeyen merged commit 0e13375 into AlchemyCMS:main Feb 1, 2023
@tvdeyen tvdeyen deleted the document-ingredient-settings branch February 1, 2023 12:45
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