-
Notifications
You must be signed in to change notification settings - Fork 113
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
sourcemap(s) are causing warnings #18
Comments
I see jou closed it.. How did you solve this? For some reason I don't see the fix?!? |
You didn't specify any details when created the issue first time #10. There are no steps and info about you env, bundler etc. |
about.ts: import {Component} from '@angular/core';
import {Http} from '@angular/http';
import { CHART_DIRECTIVES } from 'angular2-highcharts';
@Component({
selector: 'about',
templateUrl: 'app/components/about/about.html',
styleUrls: ['app/components/about/about.css'],
providers: [],
directives: [CHART_DIRECTIVES],
pipes: []
})
export class About {
private options: Object;
constructor(http: Http) {
this.options = {
title: { text: 'simple chart' },
series: [{
data: [29.9, 71.5, 106.4, 129.2],
}]
};
}
ngOnInit() {
}
} about.html <h3>About Component</h3>
<p>This is the about component!</p>
<chart [options]="options"></chart> warnings:
|
Note the paths in the message e.g. '../src/ChartEvent.ts'. If you look in the angular2-highcharts folder in node_modules, you'll notice 3 issues:
If you go into each of the the map files and change e.g. ... the messages go away ... and it works. |
@gevgeny We are also facing this issue. Can you please reopen it ? If you need further details i am ready help. |
try v0.4.0 |
The problem still exists in 0.4.1 with the Webpack starter. I get a lot of these warnings, for example: Cannot find source file '../src/ChartXAxisComponent.ts': Error: Can't resolve '../src/ChartXAxisComponent.ts'. |
I also still have the issue with v0.4.1, which I can fix by manually applying the patch from pevans306 / Raymice on my So I guess PR #100 is still applicable. |
Actually the PR #100 is changing the map files in dist directory, that are regenerated at build time, so I guess it's not an option. I don't understand how the map files are generated, but if they are referencing the |
workaround:
put this in webpack config file
list of warnings:
The text was updated successfully, but these errors were encountered: