Code review comment for ~steverydz/maas:fix-machine-listing-js-error

Revision history for this message
Björn Tillenius (bjornt) wrote :

I can confirm that there's no error shown in the JS console with this branch.

However, I'm curious as what's actually going on there.

For example, on current master, why is there no error on the initial loading of the table? And why is removeDuplicates given an undefined value?

I feel that there's an error somewhere else, and this branch would just hide it even further. Or do you have a good explanation for where the undefined value is coming from?

review: Needs Information

« Back to merge proposal