reactive-table icon indicating copy to clipboard operation
reactive-table copied to clipboard

There are multiple templates named 'leaderboard'

Open tab00 opened this issue 8 years ago • 8 comments

After updating to 0.8.43 I saw the above error in the console, and the app doesn't load.

I downgraded to 0.8.42 and my app is working again.

Please fix this problem so that we can use the latest version.

Thank you.

tab00 avatar Nov 20 '17 03:11 tab00

Sorry about that. The only thing I changed in 0.8.43 was the dependency on underscore to fix #481, and I don't see how that would cause this problem. It's commit c81598ab7b971da5baa3b7ee0ca714d123cf80ca.

Maybe meteor publish did something weird. In that case it wouldn't help to revert the change and publish again. Any ideas?

aslagle avatar Nov 20 '17 03:11 aslagle

I also saw this behavior and was unable to build render my application correctly because of issues with the leaderboard template: https://www.dropbox.com/s/q59392gj6zzv6cf/Screenshot%202017-11-19%2023.04.18.png?dl=0 reverting made it work normally

harrisonhunter avatar Nov 20 '17 04:11 harrisonhunter

Weird. The leaderboard template is used in examples but they're not supposed to be part of the package. I published with a newer version of meteor than I've used in the past so maybe it included those files even though they're not listed in package.js. I haven't been keeping up with the changes to meteor.

aslagle avatar Nov 20 '17 04:11 aslagle

Can confirm, same issue with multiple templates with one name. Switched back to 0.8.42, everything works fine. Maybe changes in build process.

EDIT Having no other ideas I tried replacing old api.add_files with api.addFiles etc (as suggested in docs). Without any success. For now I ended up with removing the whole examples dir.

@s7dhansh thought the same about eager loading, but what's the point of adding files explicitly then...

rmuratov avatar Nov 20 '17 11:11 rmuratov

I think the leaderboard template is being compiled multiple times as it is not in imports directory. The new meteor version for packages would have become more in sync with the normal apps, by eagerly loading everything in non-imports.

s7dhansh avatar Nov 20 '17 12:11 s7dhansh

I published 0.8.44 with an older version of meteor to see if it fixes the problem. I didn't make any other changes. Let me know if it works.

aslagle avatar Nov 20 '17 13:11 aslagle

Yes it does. The meteor shell issue is also fixed.

s7dhansh avatar Nov 20 '17 13:11 s7dhansh

Yes, that worked for me as well. Thanks!

rmuratov avatar Nov 20 '17 13:11 rmuratov