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

New: Add setting to include additional paths for css import path … #40

Merged

Conversation

c4ll-m3-j4ck
Copy link

This adds a setting to default.yaml that enables the integrator to add additional paths that can be looked-up and removed from css resolve() calls.

postcssOptions:
  additionalPackagePathPrefixes:
    - Sites # could be any path below /DistributionPackages

There is no default path provided and if omitted, this setting does not change any current behavior.

This would enable you to place your custom Neos-Packages in a Subfolder to keep them more organized.
A folder structure could look like this now:

DistributionsPackages
  /Sites
    /MyProject.Site
  /Plugins
    /MyProject.MyPlugin

Let me know, if you have any questions, I'd be happy to talk about this!

@jonnitto jonnitto changed the title FEATURE: Add setting to include additional paths for css import path … New: Add setting to include additional paths for css import path … Oct 10, 2023
@jonnitto
Copy link
Member

@c4ll-m3-j4ck thank you a lot for your PR. Could you also explain this behavior in the Readme?

@c4ll-m3-j4ck c4ll-m3-j4ck force-pushed the feature/configure-package-subpaths branch from 69e0b23 to ed199cb Compare October 11, 2023 06:43
@c4ll-m3-j4ck
Copy link
Author

I have added a section for the resolve() function which also explains the usage of this new setting, do you think that this is enough?

Readme.md Outdated Show resolved Hide resolved
Readme.md Outdated Show resolved Hide resolved
@jonnitto jonnitto merged commit cbd0ff6 into CarbonPackages:main Oct 11, 2023
1 check passed
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