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

Add support for location in audit #253

Closed
yanokwa opened this issue Jan 29, 2019 · 3 comments · Fixed by #260
Closed

Add support for location in audit #253

yanokwa opened this issue Jan 29, 2019 · 3 comments · Fixed by #260

Comments

@yanokwa
Copy link
Contributor

yanokwa commented Jan 29, 2019

Building on the issue at getodk/xforms-spec#219 and PR at https://github.com/opendatakit/xforms-spec/pull/221/files, we should add support for locations in the audit log.

My rough proposal is

  • Add support for three parameters for audit questions (location-priority, location-min-interval, location-max-age) (note, we don't include the odk:)
  • If one of the parameters are entered, then all three are required
  • priority has four potential values: no-power, low-power, balanced, or high-accuracy
  • max-age and min-interval are positive integers. max-age >= min-interval

Sample output looks like:

<bind nodeset="/orx:meta/orx:audit" type="binary" 
           odk:location-priority="balanced" odk:location-min-interval="10" odk:location-max-age="60" />
@yanokwa
Copy link
Contributor Author

yanokwa commented Jan 29, 2019

@KeynesYouDigIt This something you'd be willing to take on?

@ukanga
Copy link
Contributor

ukanga commented Feb 19, 2019

This is closed by #260

@KeynesYouDigIt
Copy link
Contributor

@yanokwa Sorry about that! missed your mention somehow.

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 a pull request may close this issue.

3 participants