-
Notifications
You must be signed in to change notification settings - Fork 46
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
Error: Unexpected value 'undefined' exported by the module 'IntercomModule' #14
Comments
Thanks for opening an issue! Could you please share a call stack with me? That will help me diagnose the issue much faster. Thanks! |
I'm going to close this issue in a few days if I don't get your response @Pos1tron |
Sorry for the late reply. You can go ahead and close. I got by this by
going back to an older version.
…On Wed, Aug 16, 2017 at 10:40 PM, Wilson Hobbs ***@***.***> wrote:
I'm going to close this issue in a few days if I don't get your response
@Pos1tron <https://github.com/pos1tron>
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#14 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAjzi9sPphsQzqajQtVoVDl6UHa5RJW_ks5sY6gZgaJpZM4O3rPt>
.
--
|
So the current version works as expected @Pos1tron |
@wbhob I'm getting the same error with version 1.0.0-alpha6 published on npm.
I'm importing the module (using IntercomModule.forRoot({app_id: 'appid'})) in my AppModule. When the application loads, the error from above is thrown. Any ideas? |
@wbhob I found the issue and submitted a pull request with the fix :) |
Fixed issue with circular dependency leading to error described in #14
Thank you @flore2003! Released version 1.0.0-beta.0. |
I'm using the alpha branch. Importing the module like so:
IntercomModule.forRoot({ app_id: '<%= INTERCOM_ID %>'})
The text was updated successfully, but these errors were encountered: