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

Expose orderBy as a parameter when listing documents #590

Open
michaelcho opened this issue Dec 15, 2023 · 0 comments
Open

Expose orderBy as a parameter when listing documents #590

michaelcho opened this issue Dec 15, 2023 · 0 comments

Comments

@michaelcho
Copy link

Is your feature request related to a problem? Please describe.
I would like to expose the list of spreadsheets to the user with a different orderBy than the default recency which is hardcoded here.

Describe the solution you'd like
Either:

  1. the orderBy parameter is passed down through the client methods for spreadsheet_titles() and spreadsheet_ids()
  2. OR at the very least, change the hardcoding to recency desc. I think most users would want to see the most recent files rather than the oldest files?

Any other info
I think I can make a pull request for this enhancement, so let me know if this would be accepted and I can tidy up the tests etc. Otherwise I'll just leave it on my forked version of the repo.

Thanks!

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

No branches or pull requests

1 participant