getodk / briefcase

ODK Briefcase is a Java application for fetching and pushing forms and their contents. It helps make billions of data points from ODK portable. Contribute and make the world a better place! ✨💼✨
https://docs.getodk.org/briefcase-intro
Other
60 stars 154 forks source link

Add GUI support for the ability to export a date range #117

Closed shivam-tripathi closed 7 years ago

shivam-tripathi commented 7 years ago

The ability to export a date range is something that's in the CLI but not in the GUI. It would be great to have this feature included in GUI too.

yanokwa commented 7 years ago

@shivam-tripathi Is this an issue you can take a look at for the next release?

shivam-tripathi commented 7 years ago

@yanokwa Sure thing.

yanokwa commented 7 years ago

@shivam-tripathi You still interested in contributing this feature?

shivam-tripathi commented 7 years ago

@yanokwa I am. Just a little busy at the moment.

yanokwa commented 7 years ago

We have a code freeze coming up next week and I want to make sure we have time to test and review. Should we bump to the next milestone?

shivam-tripathi commented 7 years ago

Next week sounds too close. I am afraid we will have to bump it to the next milestone.

yanokwa commented 7 years ago

Hi @shivam-tripathi! We've finished the June milestone and have started work on the July milestone. Do you have any sketches of the UI you had in mind that you can share?

yanokwa commented 7 years ago

Hi @shivam-tripathi! Since I haven't heard, back I'm tagging this as contributor-friendly to give others a chance at it.

@daveycrockett You seem to be pretty good at adding UI-y features to Briefcase. Any interest in this one?

thedavidmccann commented 7 years ago

Thanks for pointing this one out, @yanokwa! I'm happy to take it on (at the risk of pigeon-holing myself!) It will probably take me til end of next week.

batkinson commented 7 years ago

Hi all. I am currently supporting active data collection efforts using ODK in Equatorial Guinea. After taking a look at a few approaches, it appears that this functionality is a linchpin of sorts for an important workflow. As such, I think this would be great to incorporate, even if not perfect from the start. That workflow:

This last step is where briefcase's functionality is essential:

The workflow is almost complete. Fortunately briefcase appends when exporting the same forms multiple times. My guess is that this is because the command line export with date range attempts to support this type of workflow. However, without the ability to export a date range from the GUI, all you get when exporting multiple times is complete duplication of all submissions in the same CSV file.

Adding this functionality completes this workflow.

lognaturel commented 7 years ago

Thanks for describing the use case, @batkinson! That makes a lot of sense.

We'd love to get this in the August release and @daveycrockett has something that's really close in #139! @dcbriccetti is working with him on an internal review pass. I've verified that the functionality works as you describe but would be great if you could take it for a quick spin and verify that it works for you.

yanokwa commented 7 years ago

Closed by #139

batkinson commented 7 years ago

@lognaturel we have been running a local build with this PR integrated for about a week and, so far, things appear to be working just fine. The note above was really just to clarify how important this is for at least the workflow we're using. We're pleased that this will be part of a public release soon. I am happy to take this for a spin if/when it helps.

lognaturel commented 7 years ago

Awesome, thanks @batkinson!