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

Build warnings under Angular 17 #537

Open
russelg opened this issue Nov 16, 2023 · 0 comments
Open

Build warnings under Angular 17 #537

russelg opened this issue Nov 16, 2023 · 0 comments

Comments

@russelg
Copy link

russelg commented Nov 16, 2023

angular-froala-wysiwyg/froala v4.1.3.

I have been seeing this warning when building my angular projects that use Froala:

▲ [WARNING] This case clause will never be evaluated because it duplicates an earlier case clause [duplicate-case]

    angular:script/global:scripts.js:10:622815:
      10 │ ...tFileIcon("jpegIcon");case"image/jpeg":return k.icon.getFileIco...
         ╵                              ~~~~~~~~~~~~
  The earlier case clause is here:

    angular:script/global:scripts.js:10:622760:
      10 │ ...etFileIcon("gifIcon");case"image/jpeg":return k.icon.getFileIco...
         ╵                              ~~~~~~~~~~~~

The code in question it is complaining about comes from Froala (specifically the filesManager plugin):
image

Not sure if this report belongs in the angular repo, or the editor repo (https://github.com/froala/wysiwyg-editor).
However, this warning was not visible until we updated to angular 17 (and therefore esbuild as the builder).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
1 participant