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

[WIP] assets: re-structure assets and add webpack entries #133

Closed
wants to merge 1 commit into from

Conversation

ppanero
Copy link
Member

@ppanero ppanero commented Dec 4, 2020

Closes #109
Blocked by #124 (in order not to move files that will be changed...)

Problems and attempted solutions:

  • Files are fully duplicated 3 times:
    • assets/bootstrap3
    • assets/semantic-ui
    • static/

Static should only contain static content (images, cmaps, locales) but no JS/CSS files. Those should go to the assets folder. A first stab to this (adding the corresponding entry points in the webpack.py and changing imports was done.

Questions

  • How do we deal with common files?
    • CSS, are inside the scss folder (even in semantic-ui, where it should be less). Nonetheless, the content seems compatible with both. Could it go simply to the root? Or just duplicate and change file extension?

@lnielsen
Copy link
Member

lnielsen commented Dec 9, 2020

What's the status with this one? Should it be integrated?

@ppanero
Copy link
Member Author

ppanero commented Dec 9, 2020

It is pending an answer to the question stated on the PR description

@ppanero
Copy link
Member Author

ppanero commented Mar 24, 2022

has this been solved? can it be closed? I see the master tests are passing but not sure whats the semantic/bootstrap status

@lnielsen lnielsen closed this Mar 28, 2022
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.

webpack: missing bundles
2 participants