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

Reference areas from a log #106

Closed
mstenta opened this issue Oct 2, 2018 · 3 comments
Closed

Reference areas from a log #106

mstenta opened this issue Oct 2, 2018 · 3 comments

Comments

@mstenta
Copy link
Member

mstenta commented Oct 2, 2018

Logs in farmOS can be linked to farm areas. For example: an observation of pest pressure in a specific field.

This feature request is to add the ability to reference areas from a log.

There are a few considerations to figure out with this:

  • We will need to fetch a list of existing areas from farmOS and store them locally, so that they can be presented as available options to choose from. This will require a separate AJAX request - probably when the app is first initialized (after login credentials are saved for the first time). These areas will need to be stored locally so that they are available as options when you are offline. There will also need to be some way of auto-updating them from time to time - because they may change in farmOS (outside of the app). This may need to be a separate issue in itself: "Maintain a list of farm areas internally"
  • The "widget" for selecting the areas in the log form will need some thought. A simple HTML multi-select field type would be an easy first step, but if you have a large list of areas it can become unwieldy. I say we start with that, though, and create a followup issue if/when necessary. The mobile UI for multi-select is actually pretty good (easier than holding down "Shift" on a desktop, IMO).
@mstenta
Copy link
Member Author

mstenta commented Oct 2, 2018

Broke out the first bullet point to a separate issue: #23

That is really a prerequisite of this feature request.

@jgaehring
Copy link
Member

Resolved as of 10616fa

@mstenta mstenta transferred this issue from farmOS-legacy/farmOS-client Feb 19, 2019
@mstenta
Copy link
Member Author

mstenta commented Feb 19, 2019

(Transferring all issues from old repository. See #92)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

2 participants