-
Notifications
You must be signed in to change notification settings - Fork 15
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
Occurrence cube downloads #1978
Comments
Nice!! Here's some feedback. Download page
Modal
SQL editor
|
Thanks @peterdesmet
I agree it is nicer, it is only because it is makes life easier for translators. Having them write markdown with variables have caused issues in the past. |
Yeah I know those are wrong. I'm waiting for you or Matt to tell me what they should be please. I've changed the MGRS as you specified above EXTENDED_QUARTER_DEGREE_GRID should be? |
I've added mock help texts to all fields and added 2 mock articles (one for sql download and one for cubes). Help texts
Articles
Known API bugs
Download pages
That is just one idea. Other ideas for how to make the transition easier for users are welcome Other
|
Thanks. I think the text helps in guiding the user. I think adding the ability to give it a human readable type / description would be good. Alternatively, we could introduce a cube download format in the API itself, which takes the form parameters but does the SQL conversion behind the backend API. The reason to do that, would be to display to the parameters used on the download page which is shown from the DOI. A user could still "open this in the SQL builder" before submitting to do more complicated queries, but it'd hide SQL completely for anyone who didn't. I don't know what would be the more scalable option. |
|
On 4. please see our proposed approach here. A JSON object (the context) would hold the submitted form parameters. |
ping @peterdesmet and @MattBlissett
I've started the work of adding a new download option
And an SQL ui for downloads
Could you please help evaluate if this is functionally what you had in mind?
Functionally there are 2 things that aren't working
WHERE
isn't filled. Waiting for feat: internal service to translate predicates to SQL WHERE occurrence#356If the functions are as you expected, then we could think about usability (bearing in mind that we will have to reimplement this in the recent future)
The text was updated successfully, but these errors were encountered: