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

Use public folder #892

Merged
merged 4 commits into from
Feb 10, 2022
Merged

Use public folder #892

merged 4 commits into from
Feb 10, 2022

Conversation

sampoyigi
Copy link
Member

No description provided.

Signed-off-by: Sam Poyigi <6567634+sampoyigi@users.noreply.github.com>
Signed-off-by: Sam Poyigi <6567634+sampoyigi@users.noreply.github.com>
@sampoyigi sampoyigi added this to the 4.0 milestone Feb 9, 2022
@sampoyigi sampoyigi changed the base branch from develop to 4.0 February 10, 2022 17:53
@sampoyigi sampoyigi merged commit 7e509cd into 4.0 Feb 10, 2022
@sampoyigi sampoyigi deleted the use-public-folder branch February 10, 2022 17:54
@sampoyigi
Copy link
Member Author

Related #771

@sampoyigi sampoyigi mentioned this pull request Feb 10, 2022
33 tasks
@ryanmitchell
Copy link
Member

So what does this mean for assets for extensions? Do we need to provide a way of publishing them?

@sampoyigi
Copy link
Member Author

We combine assets by default, remove option to disable combiner, this way all styles and scripts can be added using addCss or adJs combined and accessed from example.com/_assets and /admin/_assets

sampoyigi added a commit that referenced this pull request Feb 25, 2022
* wip

Signed-off-by: Sam Poyigi <6567634+sampoyigi@users.noreply.github.com>

* wip

Signed-off-by: Sam Poyigi <6567634+sampoyigi@users.noreply.github.com>
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.

None yet

2 participants