-
Notifications
You must be signed in to change notification settings - Fork 127
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 : 1115/i18n : Upgrade ngx-moment + Fix Function expressions are not sup… #1197
WIP : 1115/i18n : Upgrade ngx-moment + Fix Function expressions are not sup… #1197
Conversation
…ported in decorators
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.
Hi @cedricjimenezst thanks a lot for the PR, I will test it in the next days!
(And I'll be glad to work on #1115 with you if you're interested.)
Concerning this PR, it is no big deal but could you refactor it slightly (branch name, commit message, signed commit) to fit project conventions please? (https://github.com/theopenconversationkit/tock/blob/master/CONTRIBUTING.md)
Please let me know if you have questions.
@francoisno can I merge this MR ? |
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.
Once the ng2-charts
import is removed from NLP Admin, every works well.
I will rebase, amend the commit and merge manually the PR.
Thx again.
@@ -36,6 +36,11 @@ import {NbThemeModule, NbRouteTabsetModule, NbCardModule, NbButtonModule, NbSele | |||
import {ThemeModule} from "../theme/theme.module"; | |||
import { NgxEchartsModule } from 'ngx-echarts'; | |||
import {NgbModule} from '@ng-bootstrap/ng-bootstrap'; | |||
import {ChartsModule} from "ng2-charts"; |
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.
This import/module breaks the build (at least after rebasing from master).
Merged manually. |
…ported in decorators