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

No ID provided. An ID must be given when creating a grid. #14

Open
marchrius opened this issue Oct 18, 2016 · 6 comments
Open

No ID provided. An ID must be given when creating a grid. #14

marchrius opened this issue Oct 18, 2016 · 6 comments

Comments

@marchrius
Copy link

marchrius commented Oct 18, 2016

This error occur when using ui-grid and angular-widget-grid in same project.

The Grid factory will be replaced by ui.grid.Grid

UPDATE:
If I load the widgetGrid befaore the ui.grid all works.

@W-prog
Copy link

W-prog commented Jan 18, 2017

Hello,
I have the same error even if I load the ui-grid after the angular-widget-grid
Have you an issue ?

@marchrius
Copy link
Author

@W-prog no, I solved the issue by loading widgetGrid before ui.grid.

It will be better if the maintainer changes the object name with a prefix or something else.

@W-prog
Copy link

W-prog commented Jan 19, 2017

Oh yeah, i've trying load widget-grid before but the error is more large
for example :
TypeError: Cannot read property 'length' of null
at ui-grid.js:3277

Yes for "It will be better if the maintainer changes the object name with a prefix or something else"
Have you an idea where in librairie I do that ?
I see on line 161

vm.grid = new Grid($scope.rows, $scope.columns);
For rename the var Grid but i think i do rename all on the lib

@marchrius
Copy link
Author

Yes, you have to replace all the occurrences of "Grid" into the lib.

@sebius8780
Copy link

Hello guys !
Thanks for the tip, we actually needed to change Grid to someNameGrid AND gridUtil to someNameGridUtil.

Just saying in case someone needed it !

@marchrius
Copy link
Author

@W-prog @sebius8780 At the moment I've found a work around for this (I forgotten it).

#13 In this PR there is a commit about force inject the correct Grid.

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

3 participants