Skip to content

[DEL] agrios_kobo: agrios dependency#44

Open
thomaspaulb wants to merge 1 commit intoadvanceinsight:18.0from
Therp:18.0-delete-agrios-dependency
Open

[DEL] agrios_kobo: agrios dependency#44
thomaspaulb wants to merge 1 commit intoadvanceinsight:18.0from
Therp:18.0-delete-agrios-dependency

Conversation

@thomaspaulb
Copy link
Copy Markdown

@thomaspaulb thomaspaulb commented Jan 18, 2026

The agrios module which is being depended on, does not exist.

I couldn't identify other AgriOS modules that this should depend on, there seem to be no models or views inherited from other AgriOS modules.

@thomaspaulb thomaspaulb requested a review from NL66278 January 18, 2026 21:01
@NL66278
Copy link
Copy Markdown
Contributor

NL66278 commented Jan 19, 2026

@thomaspaulb But there are still factual dependencies on agrios, like the use of the farmer.training model. Also there is a dependency on farmer.plot, that now should become model geospatial.plot, which then would need a depends on geospatial_plot.

@thomaspaulb
Copy link
Copy Markdown
Author

@NL66278 So where is farmer.training defined? Then we could depend on that specific module, as well as geospatial_plot.

@NL66278
Copy link
Copy Markdown
Contributor

NL66278 commented Jan 19, 2026

@thomaspaulb it is defined in agrios_training (which in turn depends on agrios_farmer). Actually there is a lot to be said for OCA'izing the agrios_training module as well, as giving training, organizing groups of people to be trained, giving out certificates for training followed (or examinations passed) does not seem very specific for farmers...

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

Successfully merging this pull request may close these issues.

2 participants