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

adf-content-metadata-card configuration for excluded properties ignores all but 1 configuration #9307

Closed
StephenDunn opened this issue Feb 7, 2024 · 0 comments · Fixed by #9309
Labels

Comments

@StephenDunn
Copy link
Contributor

  • New feature request
  • [x ] Bug
  • Support request
  • Documentation

Current behaviour:
When applying a configuration through an extension the configuration 'features>content-metadata-presets>custom>exclude' is ignored. Only the excluded properties from a single configuration take effect.

Expected behavior:
All excluded properties from all configurations are applied.

Steps to reproduce the issue:
Create a new extension for ADW with a config file. Alternatively add a new configuration file to be included that will be applied.
Add the following config:

{
  "features": {
    "content-metadata-presets": [
      {
        "custom": [
          {
            "includeAll": true,
            "exclude": [**Include properties/aspects to be excluded**]
          }
        ]
      }
    ]
  }
}

The exclude in the above config will not be applied.

Component name and version:
alfresco-ng2-components content-metadata 6.6.0

Browser and version:
all

Node version (for build issues):
n/a

New feature request:
n/a

StephenDunn pushed a commit to StephenDunn/alfresco-ng2-components that referenced this issue Feb 7, 2024
StephenDunn pushed a commit to StephenDunn/alfresco-ng2-components that referenced this issue Feb 7, 2024
StephenDunn added a commit to StephenDunn/alfresco-ng2-components that referenced this issue Feb 7, 2024
@DenysVuika DenysVuika added the bug label Feb 7, 2024
DenysVuika pushed a commit that referenced this issue Feb 16, 2024
…ration to include all excluded properties (#9309)

* fix filterExcludedPreset to include all configurations instead of only the first with exclusions. (#9307)

* New test for property exclusion in layout-oriented-config.service (9307)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
2 participants