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
[Vue] remove use of angularjs for expression parsing #19492
Conversation
If you don't want this PR to be closed automatically in 28 days then you need to assign the label 'Do not close'. |
This issue is in "needs review" but there has been no activity for 7 days. ping @matomo-org/core-reviewers |
This issue is in "needs review" but there has been no activity for 7 days. ping @matomo-org/core-reviewers |
This issue is in "needs review" but there has been no activity for 7 days. ping @matomo-org/core-reviewers |
This issue is in "needs review" but there has been no activity for 7 days. ping @matomo-org/core-reviewers |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Did some basic testing in the UI for the components that are affected by the changes. Couldn't find any issues.
Btw. @diosmosis we might need to update our LEGALNOTICE
and add vue.js as well as math.js maybe
@sgiehl updated the notice |
* try replacing angularjs expression parsing w/ mathjs/number * logical operator fixes * only import what is needed for math.js * remove some functionality * optimize math.js use * built vue files * update legal notice Co-authored-by: sgiehl <stefan@matomo.org> Co-authored-by: sgiehl <sgiehl@users.noreply.github.com>
* try replacing angularjs expression parsing w/ mathjs/number * logical operator fixes * only import what is needed for math.js * remove some functionality * optimize math.js use * built vue files * update legal notice Co-authored-by: sgiehl <stefan@matomo.org> Co-authored-by: sgiehl <sgiehl@users.noreply.github.com>
* try replacing angularjs expression parsing w/ mathjs/number * logical operator fixes * only import what is needed for math.js * remove some functionality * optimize math.js use * built vue files * update legal notice Co-authored-by: sgiehl <stefan@matomo.org> Co-authored-by: sgiehl <sgiehl@users.noreply.github.com>
Description:
Changes:
We're only using the smallest portion of math.js possible to support what fields need, but it still adds quite a bit to the bundle size.
Looking through the bundled code, it seems there's still quite a bit that's not needed. It's possible it could be excluded by directly including only what we need from individual math.js files, but I'm not sure it will have any effect in the end.
There are a handful of other expression parsing libraries, but all of them seem to be very old and unmaintained, except math.js.
Review