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

Issue with aurelia/binding typings #736

Closed
dhanapalschandran opened this issue Feb 4, 2019 · 12 comments
Closed

Issue with aurelia/binding typings #736

dhanapalschandran opened this issue Feb 4, 2019 · 12 comments

Comments

@dhanapalschandran
Copy link

dhanapalschandran commented Feb 4, 2019

image 6
Hi Team,

After the recent commit we are getting the build issue related to Disposable, Callable interface in typings, which was expected in aurelia templating,
Now once we fall back to 2.1.7 version, it works fine.
Kindly let us know when it will be resolved.

@bigopon
Copy link
Member

bigopon commented Feb 4, 2019

@dhanapalschandran Thanks for filing this. Can you help paste the full error message? It seems important parts of the error were not captured.

@dhanapalschandran
Copy link
Author

@dhanapalschandran Thanks for filing this. Can you help paste the full error message? It seems important parts of the error were not captured.

Hi @bigopon , I have provided the screenshot of the error we got during gulp build, Thanks.

@bigopon
Copy link
Member

bigopon commented Feb 4, 2019

@dhanapalschandran I see the updated picture. I'm curious about your usage, as it seems you are importing bindingMode from aurelia-templating?
As far as I know, it should have never worked because it's the import from aurelia-binding. The only module that reexport bindingMode is aurelia-framework, which means:

// not working
- import { bindingMode } from 'aurelia-templating';
// working
+ import { bindingMode } from 'aurelia-framework';

@dhanapalschandran
Copy link
Author

dhanapalschandran commented Feb 4, 2019

@dhanapalschandran I see the updated picture. I'm curious about your usage, as it seems you are importing bindingMode from aurelia-templating?
As far as I know, it should have never worked because it's the import from aurelia-binding. The only module that reexport bindingMode is aurelia-framework, which means:

// not working
- import { bindingMode } from 'aurelia-templating';
// working
+ import { bindingMode } from 'aurelia-framework';

Yes correct, this was error was popped from aurelia-templating index.d.ts, when i change the aurelia-binding version to 2.1.7, the error disappears

@bigopon
Copy link
Member

bigopon commented Feb 4, 2019

@dhanapalschandran If possible, can you help do a clean install of project dependencies to see if anything interesting happens? It shouldn't matter, but since we've had issues related to it before, a try may not hurt.

@josiahhaswell Can you help confirm if this is the same issue you mentioned on Gitter?

@bigopon
Copy link
Member

bigopon commented Feb 4, 2019

@dhanapalschandran Also can I know your typescript version?

@dhanapalschandran
Copy link
Author

@dhanapalschandran If possible, can you help do a clean install of project dependencies to see if anything interesting happens? It shouldn't matter, but since we've had issues related to it before, a try may not hurt.

@josiahhaswell Can you help confirm if this is the same issue you mentioned on Gitter?

Hi @bigopon , I have done clean install several times before posting the issue

@dhanapalschandran
Copy link
Author

@dhanapalschandran Also can I know your typescript version?

"typescript": "^2.8.1",

@fkleuver
Copy link
Member

fkleuver commented Feb 4, 2019

@dhanapalschandran Are you somehow directly installing from the master branch? We are moving towards a new, more streamlined workflow with CircleCI but it seems the build process is not entirely correct yet. The dist files do seem to be missing stuff. Will fix this asap

@jmezach
Copy link

jmezach commented Mar 29, 2019

I think that d359e83 might have broken this. For example, with aurelia-binding 2.2.0 I can import the disableConnectQueue function from aurelia-binding, but with 2.2.1 I cannot, even though it's available at runtime. This seems like a regression.

@bigopon
Copy link
Member

bigopon commented Mar 29, 2019

@jmezach nice, thanks for the catch. The issue comes from the fact that typings was commited to dist file in this f4012aa while it should have been committed to typings file in src. I'll fix that soon.

@bigopon
Copy link
Member

bigopon commented Mar 31, 2019

All issues have been fixed after #746 .
It can be closed @EisenbergEffect @dhanapalschandran

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

No branches or pull requests

5 participants