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

Fix gulp build deprecated warning #6543

Merged
merged 1 commit into from
Jun 13, 2023
Merged

Conversation

sjpadgett
Copy link
Member

  • move and modify @ttskch/select2-bootstrap4-theme/src/select2-bootstrap4 to public/modified
  • remove package @ttskch/select2-bootstrap4-theme/src/select2-bootstrap4

Fixes #6542

Short description of what this resolves:

May be a better way but this was pretty straight forward. Let me know if have a different idea.

- move and modify select2 bootstrap 4 theme scss
- remove package npm select2 bs4 install
@sjpadgett
Copy link
Member Author

Full install and build < 3 minutes. We can consolidate and improve this I'm sure.

@stephenwaite
Copy link
Member

stephenwaite commented Jun 12, 2023

thought you'd get schmancy with
Screenshot from 2023-06-12 17-26-12
edit: where #6524 was headed

@sjpadgett
Copy link
Member Author

The only issue are two calc in select2 bs4 upstream so just needed to own the theme. There are no other upstream issues.
If I understand what you are asking....

@stephenwaite
Copy link
Member

another approach might be run the division sass migrator command after importing but before building if understanding how it works

@sjpadgett
Copy link
Member Author

I’m not sure Stephen. It’s only that select two bootstrap for sass that’s at issue. We could bring in another theme however we’d have to modify gulp to compile.
This was just the quickest and easiest way without having to re-factor a bunch of code.
Although I’m not married to this

@bradymiller
Copy link
Member

Since it looks like upstream will never fix this (last commit in that repo in 2021 and there is even a PR that has been submitted to fix this a long time ago :) ttskch/select2-bootstrap4-theme#77 ). So seems like simplest thing is to take it over and do what @sjpadgett did (less moving pieces than fixing on fly since this will never be fixed anyways :)).

@sjpadgett sjpadgett merged commit c8199e0 into openemr:master Jun 13, 2023
@sjpadgett sjpadgett deleted the theme_building branch June 13, 2023 04:18
@stephenwaite
Copy link
Member

freedom-roadtrip

@macomeza
Copy link
Contributor

Great job!

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.

bug: Gulp theme build deprecation warning in development mode
4 participants