search

farmOS / field-kit

A modular, offline-first companion app to farmOS.
https://farmOS.org
GNU General Public License v3.0
62 stars 39 forks source link

initializeLogs is not a function #434

Closed maoneid closed 3 years ago

maoneid commented 3 years ago

initializedLogs is not a function. It is happened in home page.

jgaehring commented 3 years ago

Thanks for reporting this, @maoneid.

Which release are you using? Is it one of the official releases on Android, iOS or on the web at https://farmos.app? Or is this in the development environment?

maoneid commented 3 years ago

I run manually on local web. But when i use farmos.app, it is similar case. In Home menu, then click task. TypeError: this.initializeLog is not a function addLog TasksWidget.vue:98 click TasksWidget.vue:1

jgaehring commented 3 years ago

Oh I see it now. Just to confirm we're seeing the same error, this is when you click on one of the specific icons to create an "Activity" or "Input" etc?

maoneid commented 3 years ago

Yes, it is look like farmModule is not loaded and i try to debug it but does not resolve it. Thank You

jgaehring commented 3 years ago

I think the problem is I forgot to add the mapVuex mixin to the widget modules here at line 99:

https://github.com/farmOS/farmOS-client/blob/f50be8121c9f4892a3c4615b9169d2469ae2d0db/src/utils/fieldModules.js#L97-L100

If so it's an easy fix. I should have a chance to test it out later today and can release a patch if it all checks out.

maoneid commented 3 years ago

Thank You. I try it and it is fix the problem.

jgaehring commented 3 years ago

Good to hear! Sorry I didn't have a chance to review this today. Feel free to open up a pull request if you feel so inclined. Otherwise I can get it fixed by Monday.

jgaehring commented 3 years ago

Quick update: I just pushed a fix for this in 684f19e. However, I may hold off deploying this to farmos.app until I have more to release. But let me know, @maoneid (or anyone else reading this), if it would be helpful to get this fix deployed sooner.

maoneid commented 3 years ago

@jgaehring , thank you for fix this. it is fix for the problem.